From d15efa7325e0c5d497b9089a01a9a23a78bfd7c6 Mon Sep 17 00:00:00 2001
From: silverwind <me@silverwind.io>
Date: Tue, 9 Jun 2026 14:23:18 +0200
Subject: [PATCH] chore: select the global error container via
 `firstElementChild`

Addresses review feedback: `firstChild` could return a leading text node;
`firstElementChild` reliably selects the wrapper `<div>`.

Assisted-by: claude-code:opus-4.8
---
 web_src/js/modules/errors.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web_src/js/modules/errors.ts b/web_src/js/modules/errors.ts
index 185f2e53a3..1f1a218b8c 100644
--- a/web_src/js/modules/errors.ts
+++ b/web_src/js/modules/errors.ts
@@ -24,7 +24,7 @@ export function showGlobalErrorMessage(msg: string, msgType: Intent = 'error', d
   if (!msgContainer) {
     const el = document.createElement('div');
     el.innerHTML = html`<div class="ui container js-global-error tw-my-[--page-spacing]"><details class="ui ${msgType} message"><summary></summary></details></div>`;
-    msgContainer = el.firstChild as HTMLDivElement;
+    msgContainer = el.firstElementChild as HTMLDivElement;
   }
 
   // merge duplicated messages into "the message (count)" format