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

(Windows only) Text in tab bar is sometimes unreadable depending on brave colors and windows accent color #22027

Closed
Opposen opened this issue Mar 31, 2022 · 21 comments · Fixed by brave/brave-core#12946
Assignees
Labels

Comments

@Opposen
Copy link

Opposen commented Mar 31, 2022

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. install 1.37.109 on windows
  2. set accent color in windows settings > Personalization > Colors to a lighter color such as green or red
  3. scroll down to Show accent color on the following surfaces and make sure that Title bars and window borders is checked
  4. launch brave
  5. go to brave://settings/appearance
  6. set Brave colors to Dark
  7. open more than one tab

Actual result:

Tab text and new tab icon is black and pretty much unreadable.
image

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 is Dark and accent color is a lighter color, or when Brave colors is Light and accent color is a darker color

Brave version (brave://version info)

Brave 1.37.109 Chromium: 100.0.4896.60 (Official Build) (64-bit)
Revision 6a5d10861ce8de5fce22564658033b43cb7de047-refs/branch-heads/4896@{#875}
OS Windows 10 Version 21H1 (Build 19043.1586)

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 102.0.4977.0 side by side with how it looks in brave 1.37.109:
image

With a darker color the text becomes white:
image

However, it becomes unreadable in the light theme:
image

@aka13-404
Copy link

aka13-404 commented Apr 1, 2022

image
Can confirm, same thing happened to me as well.
Windows 10, 21H2 19044.1620, latest Brave version.

This make the browser unsuable for me :(

@Column01
Copy link

Column01 commented Apr 1, 2022

Same issue here:
image

@mozomoid
Copy link

mozomoid commented Apr 2, 2022

Hi there!
Yet another exemple:

Capture d’écran 2022-04-02 021947

Win10 Pro 21H2 19044.1586
Brave 1.37.109

@Meatclump
Copy link

Same here.
Windows 10 Pro
19042.1526
Brave 1.37.109
image

@francesco-ferraro
Copy link

Same issue here. Windows 10

image

@wbogocki
Copy link

wbogocki commented Apr 3, 2022

Likewise, Windows 11.

@nhantrn
Copy link

nhantrn commented Apr 4, 2022

Same.

Active window:
2022-04-04_131906

Inactive window:
2022-04-04_131916

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.

What it should be like, Edge browser as an example:
2022-04-04_132340

@Kaev
Copy link

Kaev commented Apr 5, 2022

Same issue here. Dark theme with gray buttons shows slightly darker gray text on inactive buttons.
Windows 11 and newest brave version.

gurnec added a commit to gurnec/brave-core that referenced this issue Apr 6, 2022
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 <pkasting@chromium.org>
Reviewed-by: Thomas Lukaszewicz <tluk@chromium.org>
Commit-Queue: Allen Bauer <kylixrd@chromium.org>
Cr-Commit-Position: refs/heads/main@{#967904}
@rebron rebron added priority/P2 A bad problem. We might uplift this to the next planned release. OS/Windows QA/Yes release-notes/include labels Apr 6, 2022
@MadhaviSeelam
Copy link

MadhaviSeelam commented Apr 6, 2022

Followed above testplan and reproduced the issue with dark and light colors

Brave 1.37.109 Chromium: 100.0.4896.60 (Official Build) (64-bit)
Revision 6a5d10861ce8de5fce22564658033b43cb7de047-refs/branch-heads/4896@{#875}
OS Windows 11 Version 21H2 (Build 22000.593)

Steps:

Scenario 1: Brave colors: Dark

install 1.37.109
set accent color in windows settings > Personalization > Colors to Green
scroll down to Show accent color on the title bars ad windows borders - toggle on
launch brave
go to brave://settings/appearance
set Brave colors to Dark
open more than one tab

Actual Results: Inactive tab text is unreadable

tab bar dark
Scenario 2: Brave colors: Light

install 1.37.109
set accent color in windows settings > Personalization > Colors to Storm
scroll down to Show accent color on the title bars ad windows borders - toggle on
launch brave
go to brave://settings/appearance
set Brave colors to Light
open more than one tab

Actual Results: Actual Results: Inactive tab text is unreadable

tab bar light

@amzero
Copy link

amzero commented Apr 6, 2022

Same here

github

Windows 8.1
Brave Browser 1.37.111

@dalmo3
Copy link

dalmo3 commented Apr 7, 2022

For the record, enabling or disabling brave://flags/#pervasive-system-accent-color makes no difference.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 7, 2022

Reproduced the issue with Green/Red accent colors - Windows 10 x64 - Brave 1.37.111 Chromium: 100.0.4896.79.

Reproducible - 1.37.111 Chromium: 100.0.4896.79

Tab text is notreadable

Not Reproducible - Brave 1.36.122 Chromium: 99.0.4844.88

1 36 x _ Tab text is notreadable

system settings screen 1 system settings screen 2 Brave tabs unreadable
image image image

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

@simonhong simonhong self-assigned this Apr 8, 2022
simonhong added a commit to brave/brave-core that referenced this issue Apr 8, 2022
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.
@simonhong simonhong added this to the 1.39.x - Nightly milestone Apr 8, 2022
@rebron rebron added this to Pending review/uplift in General Apr 11, 2022
@kjozwiak
Copy link
Member

Above requires 1.37.114 or higher for 1.37.x verification.

@rebron rebron moved this from Pending review/uplift to Completed in General Apr 12, 2022
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 13, 2022

Verification PASSED on

Brave | 1.37.114 Chromium: 100.0.4896.88 (Official Build) (64-bit)
-- | --
Revision | 4450653bfa91182e85723d8f1dee64dd6ce40ed4-refs/branch-heads/4896@{#1086}
OS | Windows 10 Version 21H2 (Build 19044.1586)

  • Confirmed tab text is readable in both dark and light accent colors

Dark theme
dark theme_tab text

Light theme
Light theme_tab text

@amzero
Copy link

amzero commented Apr 13, 2022

Today my brave updated, but the bug was not fixed here, aparentely. I'm doing something wrong?

brave

Windows 8.1

Edit: The fix don't was released yet?

@LaurenWags
Copy link
Member

Hi @amzero - we're aiming to release a version with this fix tomorrow. Stay tuned!

@tomingle
Copy link

Still not working for me with today's update:

Version 1.37.114 Chromium: 100.0.4896.88 (Official Build) (64-bit)
Windows 8.1 v6.3.9600
Settings > Appearance > Brave Colours: Light

Screenshot.1948.2022-04-14.211027.mp4

Normal windows are fine, inactive tab text in incognito windows is unreadable.

@amzero
Copy link

amzero commented Apr 14, 2022

brave

Windows 8.1
Versão 1.37.114 Chromium: 100.0.4896.88 (Versão oficial) 64 bits

@gurnec
Copy link

gurnec commented Apr 15, 2022

The update did work for me.

Version 1.37.116 Chromium: 100.0.4896.127 (Official Build) (64-bit)
Windows 10 19044.1645

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).

@amzero
Copy link

amzero commented Apr 16, 2022

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,
Do we have to open a new one?

@kjozwiak
Copy link
Member

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, Do we have to open a new one?

Thanks for all the reports 👍 I managed to reproduce the issue and created #22365 as a follow up for Win 8. As per the confirmations above, looks like the issue was resolved for both Win 11 & Win 10 but is still a problem within Win 8. Assuming it's due to Win 8 using a different theming/colour implementation compared to Win 11 & Win 10. We'll try getting this fixed ASAP 👍

@rebron rebron removed this from Completed in General Apr 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment