Skip to content

ui: link to the project's GitHub from the app bar - #26

Merged
valentinps merged 1 commit into
mainfrom
feat/topbar-github-link
Sep 6, 2026
Merged

ui: link to the project's GitHub from the app bar#26
valentinps merged 1 commit into
mainfrom
feat/topbar-github-link

Conversation

@valentinps

Copy link
Copy Markdown
Owner

Adds an icon-only GitHub link at the far right of the app bar, separated from the app's own controls by a hairline.

  • Icon-only on purpose. The bar centres the search field by giving both side sections an equal share of the leftover space, which only holds while each side fits its share. A labelled pill on the right would drag the field off-centre, and would be first to break at the 1150px breakpoint where the other pills already drop their labels. At 34px square it mirrors the menu button at the left end of the bar.
  • Resting colour is --text rather than .btn-ghost's --faint, so it carries the same weight as the pill labels beside it; hover keeps the ghost treatment (--text-bright + raised ground).
  • Real anchor with target="_blank" rel="noopener", title and aria-label.

Verified in headless Chrome against a fresh dist/ build: correct href/target, hover state renders, and the ≤1150px layout still fits.

🤖 Generated with Claude Code

An icon-only link at the far right of the bar, set off from the app's own
controls by a hairline: it points somewhere else, so it should not read as
one more view to open.

Icon-only is load-bearing, not taste. The bar centres the search field by
giving both side sections an equal share of the leftover space, which only
holds while each side fits its share -- a labelled pill here would drag the
field off-centre, and it would be the first thing to break at the 1150px
breakpoint where the other pills already drop their labels. As a 34px square
it mirrors the menu button at the opposite end of the bar instead.

It rests at --text rather than .btn-ghost's --faint so it carries the same
weight as the pill labels beside it, and keeps the ghost hover.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@valentinps
valentinps merged commit 78f2238 into main Sep 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant