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

Chrome/Edge default color-scheme scrollbars #40414

Open
3 tasks done
coliff opened this issue May 21, 2024 · 2 comments
Open
3 tasks done

Chrome/Edge default color-scheme scrollbars #40414

coliff opened this issue May 21, 2024 · 2 comments

Comments

@coliff
Copy link
Contributor

coliff commented May 21, 2024

Prerequisites

Describe the issue

Edge/Chrome recently changed the way the default color-scheme works for websites root scrollbars. If a user has dark mode enabled and the site doesn't specify a color-scheme in the CSS then dark mode will be used for the root scrollbars.

REF: https://developer.chrome.com/release-notes/125#used_color_scheme_root_scrollbars

To avoid this I think we should add color-scheme: light at the top of the :root, [data-bs-theme=light]

Reduced test cases

You can see this by going to: https://getbootstrap.com/ (if you have OS-set dark mode and switch theme to light).
image
Note the dark scrollbar. Tested on Chrome 125 and Edge 125 on Windows 11 Pro.

What operating system(s) are you seeing the problem on?

Windows

What browser(s) are you seeing the problem on?

Microsoft Edge

What version of Bootstrap are you using?

v5.3.3

@coliff coliff changed the title Chrome/Edge default color-scheme Chrome/Edge default color-scheme scroolbars May 21, 2024
@julien-deramond
Copy link
Member

julien-deramond commented May 21, 2024

Thanks @coliff for the heads-up on this topic, we'll look into it precisely to understand what changed :)
It might be closed by #39295.

@coliff coliff changed the title Chrome/Edge default color-scheme scroolbars Chrome/Edge default color-scheme scrollbars May 21, 2024
@coliff
Copy link
Contributor Author

coliff commented Nov 13, 2024

Yes your draft PR does fix the issue. Nice!

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

No branches or pull requests

2 participants