Skip to content

fix(docs-site): quiet active state for sidebar, drawer, and TOC links - #297

Merged
roboclaw-bot merged 2 commits into
mainfrom
openclaw/briny-crab
Nov 1, 2026
Merged

roboclaw-bot merged 2 commits into
mainfrom
openclaw/briny-crab

Conversation

@roboclaw-bot

Copy link
Copy Markdown
Contributor

Summary

The current-page link in the docs sidebar was rendered as brand-orange, semibold text inside a filled pill, which read as heavy. This switches every "you are here" surface to one quieter treatment.

  • Sidebar / mobile drawer (quick nav + section pages): ink text at weight 500 on a faint ink tint (--docs-nav-active-bg). The brand color stays only on the quick-nav icon as a small accent.
  • Section trigger (aria-current): weight 600 → 500 to match.
  • TOC (desktop rail + compact mobile sheet): uses the same token as the sidebar.
  • Hover: sidebar links, section triggers and group summaries share --docs-nav-hover-bg, so hover stays lighter than active. The "Back" button keeps its surface hover.
  • Light theme uses slightly softer tint values (3% hover / 4.5% active) than dark (3.5% / 6%).
  • Smoke check added so the brand-colored active text does not come back.

Only scripts/docs-site/docs-layout.css and scripts/docs-site/smoke.mjs change. The base site-css.mjs skin, which the existing smoke check covers, is unchanged.

Verification

  • npm run docs:build:preview + Playwright screenshots, before and after: desktop sidebar/TOC (dark + light), mobile drawer (dark + light), mobile TOC sheet.
  • npm run docs:build:shell (15,547 pages) + node scripts/docs-site/smoke.mjs → docs site smoke ok.
  • node --test scripts/docs-site/*.test.mjs: all pass except 7 head-drift.test.mjs cases. Those fail locally because they call gh workflow run without GitHub CLI auth, which this change does not touch.

Worked on by

roboclaw-bot and others added 2 commits November 1, 2026 12:11
Replace the brand-orange bold pill on the current sidebar/drawer page with ink text at medium weight on a faint ink tint, keeping the brand color only on the quick-nav icon. TOC, section triggers, and hover states share the same tokens.

Co-authored-by: vyctorbrzezowski <51521767+vyctorbrzezowski@users.noreply.github.com>
Worked on by:
- @vyctorbrzezowski

Co-authored-by: vyctorbrzezowski <51521767+vyctorbrzezowski@users.noreply.github.com>
OpenClaw-Publication: 6b3b8f2a-749e-4ba7-b26b-74c16a62226e
@roboclaw-bot
roboclaw-bot requested a review from a team as a code owner November 1, 2026 12:16
@roboclaw-bot
roboclaw-bot merged commit 434896b into main Nov 1, 2026
6 checks passed
@roboclaw-bot
roboclaw-bot deleted the openclaw/briny-crab branch November 1, 2026 12:36
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