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

Style scrollbars in dark mode #3975

Merged
merged 1 commit into from
Oct 26, 2021
Merged

Style scrollbars in dark mode #3975

merged 1 commit into from
Oct 26, 2021

Conversation

TheOtterlord
Copy link
Contributor

Chrome (and probably other webkit based browsers) doesn't automatically darken scrollbars.

Before:

image

After:

image

@rachelnabors
Copy link
Contributor

2020_me_thanks

This looks way better. Thanks!

@rachelnabors rachelnabors merged commit 446c15c into reactjs:beta Oct 26, 2021
gaearon added a commit that referenced this pull request Oct 27, 2021
@gaearon
Copy link
Member

gaearon commented Oct 27, 2021

I reverted this because it made the layout jump when toggling dark/light theme, and also made some of the sidebar content wrap in dark mode. We still need to fix this but the fix would need to be a bit more careful.

gaearon pushed a commit that referenced this pull request Oct 28, 2021
gaearon added a commit that referenced this pull request Oct 28, 2021
gaearon pushed a commit that referenced this pull request Oct 28, 2021
gaearon added a commit that referenced this pull request Oct 28, 2021
@gaearon
Copy link
Member

gaearon commented Oct 28, 2021

Can you please resubmit if you'd like, but fix the issue? The beta is now in the beta/ folder of the main branch.

@TheOtterlord
Copy link
Contributor Author

Sure, I'll give it a go. I've not seen this issue occure. Do you know what pages seem affected? Or does it happen on a specific screen side?

@gaearon
Copy link
Member

gaearon commented Oct 30, 2021

I was seeing it when switching between dark and light mode (on any page): the switch would jump because of the width style that only applied in one of the modes.

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

Successfully merging this pull request may close these issues.

4 participants