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

Import bookmark link text in the infobar is difficult to read with the new theme colors & updated nala colors #38354

Closed
MadhaviSeelam opened this issue May 15, 2024 · 10 comments · Fixed by brave/brave-core#23801
Assignees
Projects
Milestone

Comments

@MadhaviSeelam
Copy link

Description

Found while testing #38087. For some of the profile themes with Dark background, particularly import bookmarks now... link is difficult to read

Steps to Reproduce

  1. Install 1.66.107
  2. launch Brave
  3. opened brave://settings/manageProfile
  4. select Dark blue, Midnight blue or custom blue

Actual result:

example example
image (1) image

Expected result:

Need contrasting colors?

Reproduces how often:

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release? N/A
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@rebron @aguscruiz

@aguscruiz
Copy link

Yes, should use the dark mode version of that color. Seems to be using light mode

@simonhong
Copy link
Member

Yes, should use the dark mode version of that color. Seems to be using light mode

Correct. we should consider bg color also in this case.

@simonhong simonhong self-assigned this May 16, 2024
@rebron rebron added this to On Deck in General May 16, 2024
@frypf
Copy link

frypf commented May 16, 2024

This lack of legibility doesn't just seem isolated to the specific example mention by OP - I'm using the default theme, and having just now upgraded to 1.66.110 I instantly noticed similar difficulties reading items on "brave://discards":

Screenshot 2024-05-16 at 18 51 21

The same is true of other builtin pages, eg. settings etc. Prior to upgrading, all the dark blue clickable items were a nice legible & contrasting shade of orange IIRC.
Changing brave themes doesn't affect anything on these pages - only switching my whole system to light mode makes them readable again.

@frypf
Copy link

frypf commented May 16, 2024

Ok I see now it's apparently known as the accent colour - forgive my UI ignorance. Was this supposed to have changed from the previous orange with the latest update?

@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label May 16, 2024
@Rikk
Copy link

Rikk commented May 18, 2024

This issue with color was fixed in 1.65/1.66/1.67 at some time:
#37515brave/brave-core#23048 / brave/brave-core#23054 / brave/brave-core#23055
But in 1.66 release (v1.66.110) it is happening again as described here.

@bsclifton
Copy link
Member

bsclifton commented May 20, 2024

@frypf yes - the accent color was changed. From the orange to the "blurple" (blue/purple). Thanks for flagging 😄

@Rikk thanks for the additional info! Something may have regressed with the recent Chromium 125 upgrade

@simonhong
Copy link
Member

I'll fix this after cr126 is merged into master to avoid conflict with it as cr126 has some related color value changes.

@Rikk
Copy link

Rikk commented May 21, 2024

I'll fix this after cr126 is merged into master to avoid conflict with it as cr126 has some related color value changes.

So, does this mean Brave main color will be "Hard on the Eyes Burple" for the next 3 weeks?

cc @bsclifton

@bsclifton
Copy link
Member

@Rikk no - we'll get something earlier 😄 We're just going to fix it in the Chromium 126 code branch first - then we can backport a fix to the Chromium 125 branches (Beta / Release). Thanks for your patience!

@simonhong simonhong moved this from On Deck to In progress in General May 23, 2024
General automation moved this from In progress to Completed May 23, 2024
@brave-builds brave-builds added this to the 1.68.x - Nightly milestone May 23, 2024
@stephendonner
Copy link

stephendonner commented Jun 11, 2024

Verification PASSED using

Brave | 1.68.85 Chromium: 126.0.6478.40 (Official Build) beta (64-bit)
-- | --
Revision | 8e76a669cf78bdfa933c8d569460092a01166588
OS | Windows 10 Version 22H2 (Build 19045.4474)

Followed the steps to reproduce from #38354 (comment)

Confirmed Import bookmarks.... text was readable/higher contrast, compared with previous builds

Blue Slate Gray Purple
image image image

Also confirmed brave://discards looks good

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Completed
General
  
Completed
Development

Successfully merging a pull request may close this issue.

9 participants