Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean scroll paddings in Navigation Drawer #9026

Open
ehconitin opened this issue Dec 11, 2024 · 6 comments
Open

Clean scroll paddings in Navigation Drawer #9026

ehconitin opened this issue Dec 11, 2024 · 6 comments
Assignees
Labels
scope: front Issues that are affecting the frontend side only size: short

Comments

@ehconitin
Copy link
Contributor

Current behavior

Currently, we hide the scroll in the NavigationDrawer(#8896) because, with the current implementation, it is rendered over the NavigationDrawerItems.

Image

Expected behavior

We want a clean scroll bar with correct paddings/positioning(it should not clip over NavigationDrawerItems)

@muraliSingh7
Copy link
Contributor

Hey @ehconitin I have added margin right as 9px to whole scrollable div.
Can you please tell me whether we need horizontal scrolling on overflow or it should be hidden ?

Screenshot for your reference:
Image

@ehconitin
Copy link
Contributor Author

@muraliSingh7 no horizontal scrolling!

@Bonapara
Copy link
Member

Navigation item overflow should be hidden with ellipsis indeed [...]

@muraliSingh7
Copy link
Contributor

Screenshot :
Image
Image

Is this what you expect @Bonapara @ehconitin ?

@ehconitin
Copy link
Contributor Author

Could you please create a PR so @Bonapara and I can test out the new behavior and verify if it's working as expected?

@muraliSingh7
Copy link
Contributor

@Bonapara @ehconitin I have created a PR could you please test this on your end and provide any suggestions or feedback? Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: front Issues that are affecting the frontend side only size: short
Projects
Status: 🆕 New
Development

No branches or pull requests

3 participants