[FIX] Navigation scroll when overflow

[Hitarthsheth07]

FIX #7733

Changed the max height to 100vh and overflow to scroll when the contents are overflowing.

`display: flex;
justify-content: end; might not be needed but it is kept since it was there in the past version...

[github-actions]

Welcome!

Hello there, congrats on your first PR! We're excited to have you contributing to this project.
By submitting your Pull Request, you acknowledge that you agree with the terms of our Contributor License Agreement.

Generated by 🚫 dangerJS against e5c7a5b

[greptile-apps]

PR Summary

This pull request addresses the navigation bar overflow issue by modifying the NavigationDrawer component to enable scrolling when content exceeds the screen height.

• Added max-height: 100vh and overflow: scroll to StyledAnimatedContainer in packages/twenty-front/src/modules/ui/navigation/navigation-drawer/components/NavigationDrawer.tsx
• Retained display: flex and justify-content: end properties, which may need further review for necessity
• Potential consideration: Using overflow-y: auto instead of overflow: scroll to avoid unnecessary scrollbars
• Possible improvement: Adding padding-bottom to ensure last items are fully visible when scrolling

_{1 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

style: Consider using overflow-y: auto instead of overflow: scroll to avoid unnecessary scrollbars when content doesn't overflow

[greptile-apps]

style: justify-content: end might affect the layout unexpectedly. Verify if this is necessary for the desired layout

[Bonapara]

Hi @Hitarthsheth07, thanks for raising this bug and PR! The scroll should only apply to the middle section of the navbar, which contains the navigable items. The workspace title and support button (only on the cloud version) should remain in the same position.