fix(css): remove duplicate focus styling and fix omnibox container rendering

1 file · +20 -8

M internal/ui/theme/css.go +20 -8


@@ -109,17 +109,17 @@ button.tab-button.tab-button-active {
109	109		func generateOmniboxCSS(p Palette) string {
110	110		return `/* ===== Omnibox Styling ===== */
111	111
112		-	/* Omnibox window - floating popup */
113		-	window.omnibox-window {
114		-	background-color: var(--surface-variant);
115		-	border: 0.0625em solid var(--border);
116		-	border-radius: 0.1875em;
	112	+	/* Omnibox outer container - for positioning in overlay */
	113	+	.omnibox-outer {
	114	+	/* Positioning is handled via SetHalign/SetValign/SetMarginTop in Go */
117	115		}
118	116
119		-	/* Main container */
	117	+	/* Omnibox main container - the visible popup */
120	118		.omnibox-container {
	119	+	background-color: var(--surface-variant);
	120	+	border: 0.0625em solid var(--border);
	121	+	border-radius: 0.1875em;
121	122		padding: 0;
122		-	background-color: transparent;
123	123		}
124	124
125	125		/* Header with History/Favorites toggle */

@@ -165,9 +165,21 @@ window.omnibox-window {
165	165		caret-color: var(--accent);
166	166		}
167	167
168		-	.omnibox-entry:focus {
	168	+	entry.omnibox-entry:focus,
	169	+	entry.omnibox-entry:focus-within,
	170	+	entry.omnibox-entry:focus-visible {
169	171		border-color: var(--accent);
170	172		background-color: shade(var(--bg), 1.05);
	173	+	outline-style: none;
	174	+	outline-width: 0;
	175	+	outline-color: transparent;
	176	+	}
	177	+
	178	+	/* Override any internal focus styling */
	179	+	entry.omnibox-entry > *:focus,
	180	+	entry.omnibox-entry > *:focus-visible {
	181	+	outline-style: none;
	182	+	outline-color: transparent;
171	183		}
172	184
173	185		/* Scrolled window for suggestions */

 func generateOmniboxCSS(p Palette) string {
 	return `/* ===== Omnibox Styling ===== */
-/* Omnibox window - floating popup */
+/* Omnibox outer container - for positioning in overlay */
-window.omnibox-window {
+.omnibox-outer {
-	background-color: var(--surface-variant);
+	/* Positioning is handled via SetHalign/SetValign/SetMarginTop in Go */
-	border: 0.0625em solid var(--border);
-	border-radius: 0.1875em;
 }
-/* Main container */
+/* Omnibox main container - the visible popup */
 .omnibox-container {
+	background-color: var(--surface-variant);
+	border: 0.0625em solid var(--border);
+	border-radius: 0.1875em;
 	padding: 0;
-	background-color: transparent;
 }
 /* Header with History/Favorites toggle */
 	caret-color: var(--accent);
 }
-.omnibox-entry:focus {
+entry.omnibox-entry:focus,
+entry.omnibox-entry:focus-within,
+entry.omnibox-entry:focus-visible {
 	border-color: var(--accent);
 	background-color: shade(var(--bg), 1.05);
+	outline-style: none;
+	outline-width: 0;
+	outline-color: transparent;
+}
+/* Override any internal focus styling */
+entry.omnibox-entry > *:focus,
+entry.omnibox-entry > *:focus-visible {
+	outline-style: none;
+	outline-color: transparent;
 }
 /* Scrolled window for suggestions */