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

Made Card Scrollbar useable, changed style of others to match #396

Closed

Conversation

Slind14
Copy link

@Slind14 Slind14 commented Apr 2, 2022

before it wasn't possible to interact with it due to the faded overlay. This is bad UX when there is a lot to scroll.

@felixmosh
Copy link
Owner

Can you share a screen with the problem & the solution pls?

@Slind14
Copy link
Author

Slind14 commented Apr 3, 2022

Before there was the big nav bar with a fading effect overlaying. This effect caused it to be unclickable.
This is replaced with a scrollbar that only shows when hovering the scrollable element now.

chrome_MscjAjBnGD

The bulky device one in the sidebar nav was adjusted as well to take up less space and match the style:

image
(mostly for windows)

@felixmosh
Copy link
Owner

Thank you for the PR, but custom scroll bar is a big no-no for me.
This lib size is ~70k just for this.

Due to #399, I'll just remove the fade-out effect since there is no way in CSS to detect that scroll bar appeared.

@felixmosh felixmosh closed this Apr 8, 2022
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 this pull request may close these issues.

2 participants