Skip to content

fix(docs): make sidebar navigation scrollable - #29

Merged
AMillionDriver merged 1 commit into
mainfrom
codex/docs-sidebar-scroll
Jul 21, 2026
Merged

fix(docs): make sidebar navigation scrollable#29
AMillionDriver merged 1 commit into
mainfrom
codex/docs-sidebar-scroll

Conversation

@AMillionDriver

Copy link
Copy Markdown
Owner

Summary

  • keep the AX sidebar control pinned while documentation items scroll independently
  • keep the active documentation item visible when a page loads
  • swap the close X for a panel-close icon on hover and keyboard focus
  • add docs contract checks for the scroll region and close-icon state

Root Cause

The sidebar container itself handled overflow, so long navigation lists could extend behind the Windows taskbar. A legacy mobile .is-open { display: block; } rule also overrode the constrained flex layout when the sidebar expanded.

Testing

  • Prettier check
  • Documentation syntax and catalog check
  • Root ESLint
  • Root Vite build
  • Axoloth Style and Behavior API contract checks
  • 33 accessibility tests
  • 44 visual regression tests
  • Axoloth IntelliSense compile and tests
  • Manual 900x600 viewport test in collapsed and expanded states
  • Verified the final navigation item is reachable and the sidebar header remains pinned
  • Verified no browser console errors

The Vite build retains the existing non-blocking warning for a JavaScript chunk larger than 500 kB.

@AMillionDriver
AMillionDriver marked this pull request as ready for review July 21, 2026 07:21

@AMillionDriver AMillionDriver left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool

@AMillionDriver
AMillionDriver merged commit e3be8ff into main Jul 21, 2026
7 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