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.

Sign up for GitHub

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

Jump to bottom

Scrollbar of NcAppContentList hard to catch with resizable columns #4706

Closed
mejo- opened this issue Oct 27, 2023 · 2 comments · Fixed by #4710
Closed

Scrollbar of NcAppContentList hard to catch with resizable columns #4706

mejo- opened this issue Oct 27, 2023 · 2 comments · Fixed by #4710
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working design Design, UX, interface and interaction design feature: app-content-list Related to the app-content-list component

Comments

@mejo-
Copy link
Contributor

mejo- commented Oct 27, 2023

When using resizable columns in NcAppContent, the scrollbar for NcAppContent is hard to catch with the mouse. See the screencast:

Screencast

recording1.webm

My suggestion would be to change the scrollbar width from thin to normal if the NcAppContent columns are resizeable.

@jancborchardt @nimishavijay for design team input.
@mejo- mejo- added bug Something isn't working 1. to develop Accepted and waiting to be taken care of design Design, UX, interface and interaction design feature: app-content-list Related to the app-content-list component labels Oct 27, 2023
@mejo-
Copy link
Contributor Author

mejo- commented Oct 28, 2023

By the way, this problem only exists in Firefox. In Chrome, the scrollbar is much wider.

mejo- added a commit that referenced this issue Oct 28, 2023
@mejo-
fix(NcAppContent): Set normal scrollbar width on resizable NcAppConte…
00a6af2 
…ntList

When `NcAppContentList` is resizable, a thin scrollbar is hard to catch
with the mouse. Set scrollbar width to auto in this case.

Fixes: #4706

Signed-off-by: Jonas <jonas@freesources.org>
mejo- added a commit that referenced this issue Oct 28, 2023
@mejo-
fix(NcAppContent): Set normal scrollbar width on resizeable NcAppCont…
48b0c37 
…entList

When `NcAppContentList` is resizeable, a thin scrollbar is hard to catch
with the mouse. Set scrollbar width to auto in this case.

Fixes: #4706

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- mentioned this issue Oct 28, 2023
Merged
2 tasks
@nimishavijay
Copy link

My suggestion would be to change the scrollbar width from thin to normal if the NcAppContent columns are resizeable.

Sounds good to me! :)

mejo- added a commit that referenced this issue Nov 2, 2023
@mejo-
fix(NcAppContent): Set normal scrollbar width on resizeable NcAppCont…
6d88d70 
…entList

When `NcAppContentList` is resizeable, a thin scrollbar is hard to catch
with the mouse. Set scrollbar width to auto in this case.

Fixes: #4706

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- mentioned this issue Nov 2, 2023
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working design Design, UX, interface and interaction design feature: app-content-list Related to the app-content-list component
Projects
None yet
Milestone
No milestone
2 participants
@mejo- @nimishavijay