🎨 Let the docs sidebar read as links, not boxes - #540
Merged
Conversation
.doc-nav ruled the whole stack and every item within it, so the eight entries read as a column of outlined squares rather than a nav. Hover and the current page filled a row with --panel, which in the dark palette is #201e1a against #131210 paper — the current page was close to invisible there. Drop the rules and the fills. Entries are now plain stacked links in --body, and the current page takes the accent: green text at weight 800 behind a green leading rule. That rule is transparent at rest, so marking an entry shifts no text. Hover lifts the colour to --ink and shows the rule in --line, and :focus-visible now draws an accent outline where nothing marked keyboard focus before. Sidebar width, the Documentation label, the entries and their hrefs, and the main content position are unchanged. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0125zbFPKdEhKdUJxuRufpfj
PR #540: 🎨 Let the docs sidebar read as links, not boxes1 files, +15 / -11 Scope✅ PR scope looks good. Structural✅ No structural bloat detected. Slop✅ Slop indicators look low. Static Analysis✅ Oxlint found no issues. CorrectnessNo extraneous code patterns detected. |
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
2 participants
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.
Why
The documentation sidebar presents every destination as a ruled box. That treatment makes navigation links look like controls and adds visual weight beside long-form documentation.
What changes
Before:
The sidebar is one bordered stack with dividers, padded items, inverted hover styling, and an inverted current-page item.
After:
The sidebar reads as a compact list of links. Hover and current-page states use a stable leading rule, the current page uses the green accent, and keyboard focus keeps an explicit visible outline.
How it works
The sidebar removes the enclosing and divider borders, reduces link padding, reserves a transparent left rule to prevent state changes from shifting text, and applies existing theme colors to hover, focus, and current-page states.
How to verify it
git diff --check origin/main...origin/fix/docs-sidebar-link-stylepasses, catching whitespace errors in the patch.Scope
Included
Intentionally unchanged
Risks and limitations
Scope confirmation