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

white text is being used in certain area's making it hard to read when using light theme #6573

Closed
kjozwiak opened this issue Oct 22, 2019 · 3 comments · Fixed by brave/brave-core#3801

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Oct 22, 2019

Description

As per @petemill, looks like the 'prominent' button text colour value is being used in more locations than just the 'prominent' button.

Also reported by @Brave-Matt via Slack 👍

Steps to Reproduce

Case #1

  1. launch brave (0.73.12 CR: 78.0.3904.50 in this instance)
  2. visit http://brianbondy.com and block JavaScript via Shields
  3. click on the "lock" icon via URL and notice that the JS setting is blank/using white text

Case #2

  1. launch brave (0.73.12 CR: 78.0.3904.50 in this instance)
  2. login into https://stats.brave.com and save the password
  3. once you've saved the password, logout and reload the website
  4. click on the username to auto-fill and you'll notice that the username is using white text

** Case #3 **
From #6630

Actual result:

Screen Shot 2019-10-21 at 8 43 23 PM

Screen Shot 2019-10-21 at 8 37 04 PM

Screen Shot 2019-10-21 at 8 37 10 PM

Screen Shot 2019-10-22 at 12 14 53 PM

Expected result:

Text should be black and readable when using the light theme.

Reproduces how often:

100% reproducible when using the above STR.

Brave version (brave://version info)

Brave 0.73.12 Chromium: 78.0.3904.50 (Official Build) nightly (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS macOS Version 10.14.6 (Build 18G95)

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the dev channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

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

Miscellaneous Information:

CCing @brave/legacy_qa @petemill @rebron @Brave-Matt

@LaurenWags
Copy link
Member

Marking as QA/Blocked until new build is done. This will be available in 0.71.107 and above.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 1, 2019

The fix brave/brave-core#3801 is merged in 0.71.107 hence removed QA blocked label

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 1, 2019

Verification passed on

Brave 0.71.107 Chromium: 78.0.3904.70 (Official Build) beta (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified the STR from the description

    • Case 1 - JS settings white text box is removed now
      image

    • Case 2 - saved username is displayed as expected
      image
      image

    • Case 3 - Text in find bar is visible when text is not selected.
      image
      image

Verification passed on

Brave 0.71.111 Chromium: 78.0.3904.87 (Official Build) (64-bit)
Revision 20c21f4010010f32462ea8e1d6af30cef66d48c8-refs/branch-heads/3904@{#840}
OS Ubuntu 18.04 LTS
  • Verified the STR from the description

    • Case 1 - JS settings
      image

    • Case 2 - saved username is displayed as expected
      image

    • Case 3 - Text in find bar is visible when text is not selected.
      image

Verified passed with

Brave 0.71.111 Chromium: 78.0.3904.87 (Official Build) (64-bit)
Revision 20c21f4010010f32462ea8e1d6af30cef66d48c8-refs/branch-heads/3904@{#840}
OS macOS Version 10.13.6 (Build 17G5019)
  • Verified the test cases from the description

    • Case 1 - JS settings

Screen Shot 2019-11-04 at 2 02 45 PM

  • Case 2 - saved username is displayed as expected

Screen Shot 2019-11-04 at 2 05 46 PM

Screen Shot 2019-11-04 at 2 05 52 PM

  • Case 3 - Text in find bar is visible when text is not selected.

Screen Shot 2019-11-04 at 2 08 11 PM

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