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

Mini scroll bar on each selection in filter menu #929

Closed
l2dy opened this issue Nov 9, 2022 · 6 comments · Fixed by #930
Closed

Mini scroll bar on each selection in filter menu #929

l2dy opened this issue Nov 9, 2022 · 6 comments · Fixed by #930

Comments

@l2dy
Copy link
Contributor

l2dy commented Nov 9, 2022

Regression is probably introduced in 5d77473.

double-scroll

@adrianmroz-allegro
Copy link
Contributor

What browser and OS? We don't see that on our setup.

@l2dy
Copy link
Contributor Author

l2dy commented Nov 9, 2022

Chrome 107 on macOS with mouse plugged in.

@adrianmroz-allegro
Copy link
Contributor

Damn, I have similar setup and don't see that.

Are you feeling adventurous and want to fiddle with CSS and contribute? overflow-y: hidden or something like that could work?

@l2dy
Copy link
Contributor Author

l2dy commented Nov 9, 2022

overflow-y: hidden on .string-value .value-wrapper seems to fix it.

The problem was introduced by top: 1px in .string-value .value-wrapper .checkbox. Removing it could also fix the issue.

@adrianmroz-allegro
Copy link
Contributor

Ok, I managed to reproduce it. You have "Always show scrollbars" setting in macOS. Fix should land with next version quite soon.

@adrianmroz-allegro
Copy link
Contributor

I was wondering why boolean filters don't show scrollbar but string do. Apparently, changing overflow-x affects overflow-y property default value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants