Skip to content

fix(docs-site): expose sections in mobile menu#101

Merged
vyctorbrzezowski merged 1 commit into
openclaw:mainfrom
vyctorbrzezowski:contrib/mobile-docs-section-switcher
Jul 11, 2026
Merged

fix(docs-site): expose sections in mobile menu#101
vyctorbrzezowski merged 1 commit into
openclaw:mainfrom
vyctorbrzezowski:contrib/mobile-docs-section-switcher

Conversation

@vyctorbrzezowski

Copy link
Copy Markdown
Member

Summary

  • add a compact top-level section switcher to the proposed mobile drawer
  • keep the active section's page navigation directly below it
  • simplify the proposed mobile header while preserving language and theme controls
  • cover drawer geometry, section state, focus, and responsive header behavior

Context

Thanks to @bdjben for surfacing and documenting the mobile navigation discovery gap in openclaw/openclaw#100908. This PR builds on that work by exposing the same top-level sections in the site shell on every docs page.

Proposed UI

Proposed collapsed mobile section switcher

Proposed expanded mobile section switcher

Verification

  • npm test
  • npm run docs:build:shell
  • npm run docs:smoke:shell
  • node --check scripts/docs-site/build.mjs
  • node --check scripts/docs-site/assets.mjs
  • node --check scripts/docs-site/smoke.mjs
  • node --check scripts/docs-site/visual-smoke.mjs
  • Playwright checks at 390 px and 603 px for section navigation, drawer closure, header alignment, and horizontal overflow

@vyctorbrzezowski
vyctorbrzezowski marked this pull request as ready for review July 11, 2026 23:37
@vyctorbrzezowski
vyctorbrzezowski requested a review from a team as a code owner July 11, 2026 23:37
@vyctorbrzezowski vyctorbrzezowski self-assigned this Jul 11, 2026
@vyctorbrzezowski
vyctorbrzezowski merged commit c22db64 into openclaw:main Jul 11, 2026
4 checks passed
@vyctorbrzezowski

Copy link
Copy Markdown
Member Author

Merged via squash.

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