[FIX] Sidebar Projects section scroll issue#8
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adjusts the Sidebar layout to eliminate nested scrolling in the Projects section by consolidating sidebar sections into a single scrollable container.
Changes:
- Introduces a parent
overflow-y-autocontainer intended to scroll primary navigation, workspace, favorites, and projects together. - Removes the Projects section’s internal
overflow-y-autoso it relies on the parent scroll container.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| {/* Scrollable area: primary nav, workspace, favorites, projects */} | ||
| <div className="flex min-h-0 flex-1 flex-col overflow-y-auto"> |
There was a problem hiding this comment.
Issue #7 asks for the sidebar to scroll starting from the “New work item” button, but the new scrollable container starts after that section. As a result, “New work item”/Search remains fixed while the rest scrolls. If the intent is to fully address #7, move the scrollable wrapper up to include the “New work item + Search” block (or update the issue/PR description to clarify the intended behavior).
|
well done. |
This pull request makes layout improvements to the
Sidebarcomponent, specifically to enhance scrolling behavior and structure. The main change is to ensure that the main navigation, workspace, favorites, and projects sections are all contained within a single scrollable area, providing a better user experience.Layout and scrolling improvements:
divto ensure these sections scroll together, improving usability and layout consistency. [1] [2]Closes #7