Simplify Filter Sidebar component and improve scrollbar styles #432
Conversation
Do you mean that on this branch you do not see the scrollbar track? This is a feature, not a bug here :) Since the filter sidebar doesn't scroll (it has the full height here), I thought that I might set all colors to transparent for the transitional styles. Can you see the scrollback track (ie the gap between the filters sidebar and the main content) on the main branch? |
327eca9
to
8661ab9
Compare
No, I never see it anywhere. Here's the live site for me: It's because the site doesn't restrict the sidebar height, so there's never a scrollbar! This work should be very useful however when we can switch back to a fixed 100vh-height sidebar with its own scrolling. |
Possible you have different “Show scroll bars” settings in MacOS? (https://support.apple.com/guide/mac-help/change-general-preferences-mchlp1225/mac) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lovely work; the removal of the FiltersList
is great.
As I mentioned, I now am able to see the scrollbar track when using my mac in clamshell mode with an external mouse and keyboard 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
8661ab9
to
d266896
Compare
Fixes
(slightly) Related to #412 by @panchovm
Description
This PR removes one of the nested Sidebar Filter components to simplify it, and also removes the gap between the sidebar and main content by setting the scrollbar background to transparent. The scrollbar for Filter Sidebar is complicated with the iframe: it now technically doesn't overflow, so the 'thumb' of the scrollbar is invisible, but the gap is still there.
This PR is based on the
VCheckbox
PR.Testing Instructions
Run the app and open the Filter sidebar. You should not see this gap between it and the content:
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin