[SachinD6]

Second polish round, driven by screenshots instead of guesses.

What was wrong

• Light and dark overlapped. The chrome painted hardcoded hex values that fought the theme variant, and GTK has no light-dark() function, so those pins were the only thing "setting" a scheme. There was also nothing stopping the internal pages from disagreeing with the shell.
• The hamburger menu never opened. It had 33 rows, which made it 1283 px tall. GTK silently refuses to show a popover taller than the window, so the click landed on a button that could not present anything.
• Sidebar had no breathing room, and its selected row was an accent wash that turned muddy with a warm system accent.
• The address bar was 40 px with a 20 px radius, inside a header that also drew its own padding.

What changed

• Every chrome colour now comes from a libadwaita token in a new theme module, so surfaces follow the variant by construction. A test rejects raw colours. theme::apply returns what the toolkit actually did, and the internal pages are built from that same answer, so a page can never render light inside a dark shell.
• One scheme decision, one stylesheet, one place to change chrome.
• Chrome geometry: 34 px address bar at a 9 px radius with a tinted field, 8 px header padding, 28 px tabs with a round close button and an accent underline on the active tab, and a hairline under the tab strip.
• The sidebar is 260 px and full height (it now sits beside the toolbar view, so the tab strip no longer runs across its top), with 34 px rows, 12 px insets, hover, a neutral selection plus an accent edge, an attention dot on the favicon, and hover-revealed close buttons that stay allocated so hovering never reflows a title.
• The menu is a real icon menu: 22 rows with icons, shortcut hints, and a scroller. Palette rows carry icons too.
• pages.rs now owns every internal page behind a small interface, with its own tests. main.rs drops from 6246 to 5241 lines.

Verification

• scripts/ui-shot.sh runs the real binary under Xvfb and captures PNGs; scripts/px.py samples pixels; scripts/check-css.py enforces the token rule; BR0X_SHOT, BR0X_SHOT_OPEN, and BR0X_SHOT_DUMP are documented in docs/UI-VERIFY.md.
• Sidebar surface measured: #ebebed in Light, #2e2e32 in Dark.
• The menu now opens as a 305x590 popup on click, with content captured.
• The pages extraction is pixel-identical: the start page renders to the same PNG bytes before and after (md5 c8df5f1b).
• cargo clippy --workspace --all-targets -- -D warnings, cargo fmt --check, cargo test --workspace (147 tests), scripts/check-css.py, and node --check on every inline script all pass.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: e5d8a243-6454-484a-b7af-ee844e98e7a1

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}