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

Remove scrollbars from grid stack cards but still allow scrolling #13076

Closed
jsenecal opened this issue Jul 3, 2023 · 4 comments
Closed

Remove scrollbars from grid stack cards but still allow scrolling #13076

jsenecal opened this issue Jul 3, 2023 · 4 comments
Labels
topic: UI/UX User interface or user experience related work type: feature Introduction of new functionality to the application

Comments

@jsenecal
Copy link
Contributor

jsenecal commented Jul 3, 2023

NetBox version

v3.5.4

Feature type

Change to existing functionality

Proposed functionality

Related to 725c0b5;
Currently, grid cards have scrollbars when they overflow their containers. Lests hide them where it makes sense.

Use case

These scrollbars look odd to me and consume visual space in the dashboard while not providing much functionality (scrolling with the mouse or with a finger on touch enabled devices works just fine):
image

This, combined with the grid size sometimes creates situations where a scrollbar is visible, but only because of the padding on the elements within the cards.

Adding css styles to hide the scrollbars on the appropriate classes addresses that as implemented here fixes that

Database changes

None

External dependencies

None

@jsenecal jsenecal added the type: feature Introduction of new functionality to the application label Jul 3, 2023
@jsenecal jsenecal self-assigned this Jul 3, 2023
@jsenecal jsenecal added the topic: UI/UX User interface or user experience related work label Jul 3, 2023
@stavr666
Copy link

stavr666 commented Jul 4, 2023

I'm totally support padding fix.

Examples.
Minimal widget unusable with 1 string:
image
2nd height have scrollbar with 3 strings:
image
3rd height have either space or scroll with 5-6 strings:
image

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. Do not attempt to circumvent this process by "bumping" the issue; doing so will result in its immediate closure and you may be barred from participating in any future discussions. Please see our contributing guide.

@github-actions github-actions bot added the pending closure Requires immediate attention to avoid being closed for inactivity label Oct 3, 2023
@jsenecal
Copy link
Contributor Author

jsenecal commented Oct 3, 2023

#12128 is probably blocking this

@jeremystretch
Copy link
Member

Currently, grid cards have scrollbars when they overflow their containers. Lests hide them where it makes sense.

This would be problematic from a UX perspective, as the absence of a scrollbar can obscure the existence of additional content. For instance, in your own screenshot above, removing the scrollbar would suggest that "VLANs" is the last item in the list, when there is actually more content to reveal.

@jeremystretch jeremystretch closed this as not planned Won't fix, can't repro, duplicate, stale Oct 13, 2023
@jeremystretch jeremystretch removed the pending closure Requires immediate attention to avoid being closed for inactivity label Oct 13, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic: UI/UX User interface or user experience related work type: feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

3 participants