From b38c87185abaaa92b6220f4593cca28492a62fa6 Mon Sep 17 00:00:00 2001
From: Brian Krabach <brkrabac@microsoft.com>
Date: Mon, 30 Mar 2026 07:14:24 -0700
Subject: [PATCH] =?UTF-8?q?fix:=20settings=20dialog=20always=20visible=20?=
 =?UTF-8?q?=E2=80=94=20display:flex=20overrode=20native=20<dialog>=20hidde?=
 =?UTF-8?q?n=20state?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

<dialog> is hidden by default via UA stylesheet (display:none). Our CSS
set display:flex unconditionally, overriding the UA rule and making the
dialog always visible. Fix: only apply display:flex when the dialog has
the [open] attribute (added by showModal()).
---
 muxplex/frontend/style.css | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/muxplex/frontend/style.css b/muxplex/frontend/style.css
index b031f16..c240f09 100644
--- a/muxplex/frontend/style.css
+++ b/muxplex/frontend/style.css
@@ -991,6 +991,13 @@ body {
   padding: 0;
   overflow: hidden;
   margin: 0;
+}
+
+/* <dialog> is display:none by default (UA stylesheet). When opened via
+   showModal(), the browser adds the [open] attribute. We apply flex layout
+   only when open — otherwise our display:flex would override the UA
+   display:none and make the dialog always visible. */
+.settings-dialog[open] {
   display: flex;
   flex-direction: column;
 }