feat: redesign the desktop chrome and consolidate the stylesheet - #18
Merged
Merged
Conversation
Replace the three accreted stylesheets (styles, security, cockpit) with one cascade plus a generated themes.css and a self-contained splash.css. Introduce a real type scale, a single interactive highlight, consistent radii, and a restrained glow budget. Shell: remove the redundant workspace header row and move repository actions into the titlebar; make the rail a fixed-width strip with visible labels instead of expanding over content on hover; add failure toasts with a "View logs" shortcut; list repositories and repository actions in the command palette. Code view: lint opens from a header button instead of a permanent bar; the tree toggle moves from Tab to T so focus navigation works. Fixes: placeholder when the current branch is unknown; pin the root chain to the viewport so the UI scale transform fills the window and scrollIntoView cannot scroll the page; mock GitIdentity in the Selenium harness so the suite runs again; convert synchronous bridge failures into rejections.
b404dev
force-pushed
the
feat/frontend-design-pass
branch
from
September 4, 2026 13:55
2eb800d to
a825911
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A front-end design pass aimed at hierarchy, legibility, and UX friction. Rendered through the mock harness before and after; screenshots are in the review page linked from the conversation.
Changed
styles.css,security.css, andcockpit.csshad five overlapping "passes" redefining the same selectors (the shell grid alone was declared five times) with!importantoverrides and ~40% dead rules. They are nowstyles.css(tokens → base → chrome → views → overlays),themes.css(one block per theme, generated by merging the three per-theme fragments), andsplash.css(the brand animation, moved verbatim).TabtoT.Fixed
100vh / scale²), leaving a gap at 120%. The root chain is now pinned to the viewport and clipped, soscrollIntoViewin tree navigation cannot scroll the page.GitIdentity, so the titlebar threw on startup and the suite has been red since 0.1.16. The bridge cache now also converts synchronous failures into rejections.Verification
make checkpasses (vitest, tsc/vite build, go test, vet, gofmt).npm run test:e2epasses.