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

feat: style scrollbar #135

Merged
merged 3 commits into from
Aug 25, 2024
Merged

feat: style scrollbar #135

merged 3 commits into from
Aug 25, 2024

Conversation

steveiliop56
Copy link
Contributor

I simply changed the default scrollbar to have transparent background and use the border color to match beszel's theme. Here is a preview with 175% zoom:

image

And in light mode:

image

@henrygd henrygd merged commit 6882493 into henrygd:main Aug 25, 2024
@henrygd
Copy link
Owner

henrygd commented Aug 25, 2024

Good idea, thanks. I may tweak it a bit for contrast and give it a min-width to prevent it overriding mobile scrollbars.

@steveiliop56
Copy link
Contributor Author

Yeah no problem! My ocd was just dying with the default scrollbar lol

@Simonh2o
Copy link

I'd recommend only changing the webkit scrollbars for windows/chrome OS combo and not for mac/linux/phones which utilizes floating scrollbars (prevents scrollbar from taking up space in dom). Setting the color-scheme to dark would be enough for windows OS so you get rid of the light scrollbar during dark mode. Just my opinion.

@steveiliop56
Copy link
Contributor Author

It's yours man do whatever you want lol.

@henrygd
Copy link
Owner

henrygd commented Aug 25, 2024

@Simonh2o Good point.

After doing some research, it looks like setting the property below when dark mode is enabled will apply a darker theme to the native scrollbars.

color-scheme: dark;

So I think I'll go this route instead of overriding with css.

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.

3 participants