fix(frontend): isolate sidebar scrolling across the app - #235
Conversation
9749601 to
ce3c72c
Compare
|
Follow-up improvements added in
Verified with:
Also checked both sidebar states in the local Vite app. |
|
Hey @oshuej198 thank you! Currently verifying this. Minor issue I noticed:
The missing shortcut help on the right creates a visual gap here |
|
Thanks for the feedback! I found the actual root cause of the delayed sidebar items.
|
|
This PR has merge conflicts that need to be resolved before it can be merged. Please rebase on the latest |
|
I think all the overflow-hidden on the main content areas have to be replaced, it cuts off everything that needs to grow vertically, for example a long milestone list, iteration list etc |
# Conflicts: # frontend/src/lib/features/collections/CollectionsNavigation.svelte
|
Merge conflicts have been resolved. |
|
Review complete, did a visual check and everything looks good too |







Closes #224
What changed
ScrollableSidebarandNavigationSidebarprimitives with fixed header/footer regions and independent content scrollingAdditional improvements beyond #224
While validating the shared layout, the main application sidebar received a broader UX pass:
CkeyVerification
npm run typechecknpm run checknpx vitest run— 35 tests passednpm run buildCoverage
The shared layout contract now covers expanded and collapsed main navigation, global and workspace navigation variants, independently scrolling content panes, and short-content sidebars.