-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Import bookmark link text in the infobar is difficult to read with the new theme colors & updated nala colors #38354
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
Comments
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. |
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? |
This issue with color was fixed in 1.65/1.66/1.67 at some time: |
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 |
@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! |
Verification
|
Blue |
Slate Gray |
Purple |
---|---|---|
![]() |
![]() |
![]() |
Description
Found while testing #38087. For some of the profile themes with Dark background, particularly
import bookmarks now...
link is difficult to readSteps to Reproduce
Actual result:
Expected result:
Need contrasting colors?
Reproduces how often:
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
@rebron @aguscruiz
The text was updated successfully, but these errors were encountered: