-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
(Windows only) Text in tab bar is sometimes unreadable depending on brave colors and windows accent color #22027
Comments
Likewise, Windows 11. |
Same. I think the problem is the new color scheme engine in Brave tries to match the color of Windows accent color setting, but forgot to also applies that to the main windows background color. Thus the contrast is out of whack. |
Same issue here. Dark theme with gray buttons shows slightly darker gray text on inactive buttons. |
Fixes brave/brave-browser#22027 This regression, which caused inactive tabs under certain dark/light mode conditions on Windows to have very low contrast, was cause by the Chromium 100 change below. This fix was modelled after 5c145e9. https://chromium.googlesource.com/chromium/src/+/23a6ae6a05ef9407f56c29cbbbb133463c734099 Remove use of HasCustomColor in TabStrip and use ColorProvider. Systematized all the ChromeColorIds used for the tab strip. Bug: 1292176 Change-Id: I9dbed732cd977a184c59e45d53b918774a44799d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3425646 Reviewed-by: Peter Kasting <[email protected]> Reviewed-by: Thomas Lukaszewicz <[email protected]> Commit-Queue: Allen Bauer <[email protected]> Cr-Commit-Position: refs/heads/main@{#967904}
Followed above testplan and reproduced the issue with dark and light colors
Steps: install 1.37.109 Actual Results: Inactive tab text is unreadable ![]() install 1.37.109 Actual Results: Actual Results: Inactive tab text is unreadable ![]() |
For the record, enabling or disabling |
Reproduced the issue with Green/Red accent colors - Windows 10 x64 - Brave Reproducible -
|
system settings screen 1 | system settings screen 2 | Brave tabs unreadable |
---|---|---|
![]() |
![]() |
![]() |
Conclusion: The issue got introduced in 1.37.x with Chromium: 100.0.4896.79
and the issue is Not reproducible in Brave 1.36.x with Chromium: 99.0.4844.88
cc: @rebron @bsclifton @simonhong @brave/qa-team
fix brave/brave-browser#22027 As we don't use accent color for title bar, tab color should not be affected also by accent color. Fixed by preventing Windows' accent color affect frame colors.
Above requires |
Verification PASSED on
|
Hi @amzero - we're aiming to release a version with this fix tomorrow. Stay tuned! |
Still not working for me with today's update:
Screenshot.1948.2022-04-14.211027.mp4Normal windows are fine, inactive tab text in incognito windows is unreadable. |
The update did work for me. Version 1.37.116 Chromium: 100.0.4896.127 (Official Build) (64-bit) It did not work for me on a (not updated / old) Windows 8.1 VM I gave it a quick try on, it looked like the screenshots added by the two comments directly above. Trying high-contrast themes further changed the text color (for the worse). |
I'm new in GitHub. The problem wasn't fixed on Windows 8.1. I'm not finding an opened issue to deal with it, |
Thanks for all the reports 👍 I managed to reproduce the issue and created #22365 as a follow up for |
Description
On windows, having a lighter accent color causes the text on unselected tabs, as well as the new tab icon in the tab bar to become dark and unreadable when using the default dark theme. Having a darker accent color causes the text to become unreadable in the light color theme
Steps to Reproduce
1.37.109
on windowsaccent color
inwindows settings > Personalization > Colors
to a lighter color such as green or redShow accent color on the following surfaces
and make sure thatTitle bars and window borders
is checkedbrave://settings/appearance
Brave colors
toDark
Actual result:
Tab text and new tab icon is black and pretty much unreadable.
![image](https://user-images.githubusercontent.com/82846166/161150556-8cb368d9-1afc-461a-a5d4-8cafe821c893.png)
Expected result:
Tab text and new tab icon should be a lighter color in the dark theme
Reproduces how often:
Only on when
Brave colors
isDark
and accent color is a lighter color, or whenBrave colors
isLight
and accent color is a darker colorBrave version (brave://version info)
Version/Channel Information:
The problem is also present on the current nightly release
1.39.22
Other Additional Information:
The same behavior happens on chromium, but since the top bar on chromium is set to the color of the windows accent color, the text is still readable. Since the top bar is always dark colored in the brave dark theme, the text just becomes unreadable.
How it looks in chromium
![image](https://user-images.githubusercontent.com/82846166/161153832-bc6c19ed-1ff6-496c-818b-d889d925ab75.png)
102.0.4977.0
side by side with how it looks in brave1.37.109
:With a darker color the text becomes white:
![image](https://user-images.githubusercontent.com/82846166/161153872-9c92107c-b8d2-4b9b-9b2f-7840b5016fb7.png)
However, it becomes unreadable in the light theme:
![image](https://user-images.githubusercontent.com/82846166/161153977-ca39d643-ea70-4983-9ce7-28488ca55fb4.png)
The text was updated successfully, but these errors were encountered: