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

Most webui buttons are hidden in new dark theme with c74 #3894

Closed
jasonrsadler opened this issue Mar 26, 2019 · 5 comments
Closed

Most webui buttons are hidden in new dark theme with c74 #3894

jasonrsadler opened this issue Mar 26, 2019 · 5 comments
Assignees
Labels
design A design change, especially one which needs input from the design team feature/user-interface All UI related priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes regression release-notes/exclude

Comments

@jasonrsadler
Copy link

jasonrsadler commented Mar 26, 2019

Description

When in main settings in dark mode, regular (not 'action') buttons (e.g. the 'cancel' button within modal, the 'default browser' button, etc) cannot be seen.

Test plan / Steps to Reproduce

  1. Open Brave
  2. Open settings (switch to dark theme if not already)
  3. Go to 'Clear Browsing Data'

Actual result:

'cancel' button is hidden
Screen Shot 2019-03-24 at 11 57 57 AM

Expected result:

'cancel' button should be easily seen

Reproduces how often:

easily

Brave version (brave://version info)

0.64.x (c74)

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? No

Website problems only:

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

Additional Information

cc @petemill

@bsclifton bsclifton added design A design change, especially one which needs input from the design team feature/user-interface All UI related labels Mar 26, 2019
@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Mar 29, 2019
@rebron
Copy link
Collaborator

rebron commented Mar 29, 2019

@petemill Can you take a look?

@petemill
Copy link
Member

Yes I’ll fix it when c74 lands again

@petemill
Copy link
Member

petemill commented Apr 1, 2019

This also affects normal buttons, changing title / description to reflect that

@petemill petemill changed the title 'Cancel' button in main settings is hidden in dark theme Most webui buttons are hidden in new dark theme Apr 1, 2019
@petemill petemill changed the title Most webui buttons are hidden in new dark theme Most webui buttons are hidden in new dark theme with c74 Apr 1, 2019
@bsclifton
Copy link
Member

Resolved when C74 was merged to master with #3931 😄

@LaurenWags
Copy link
Member

LaurenWags commented May 7, 2019

Verified passed with

Brave 0.64.72 Chromium: 74.0.3729.131 (Official Build) beta(64-bit)
Revision 518a41c1fa7ce1c8bb5e22346e82e42b4d76a96f-refs/branch-heads/3729@{#954}
OS Mac OS X
  • Verified test plan from description for both dark and light themes. cc @kjozwiak to check macOS using Mojave for 'matches macOS theme' option.
    Screen Shot 2019-05-07 at 4 25 44 PM
    Screen Shot 2019-05-07 at 4 26 45 PM

Verification PASSED on macOS 10.14.4 x64 using the following build:

Brave 0.64.72 Chromium: 74.0.3729.131 (Official Build) beta(64-bit)
Revision 518a41c1fa7ce1c8bb5e22346e82e42b4d76a96f-refs/branch-heads/3729@{#954}
OS Mac OS X

Screen Shot 2019-05-07 at 7 31 12 PM

Screen Shot 2019-05-07 at 7 31 27 PM

Verification passed on

Brave 0.64.72 Chromium: 74.0.3729.131 (Official Build) beta (64-bit)
Revision 518a41c1fa7ce1c8bb5e22346e82e42b4d76a96f-refs/branch-heads/3729@{#954}
OS Windows 10 OS Build 17134.523
  • Verified the STR mentioned in the description
  • Verified that the Cancel button is seen when browser theme is changed to Dark
  • Verified that the Cancel button is seen when OS theme is changed to Dark
  • Verified that buttons are shown clearly when theme is switched from Dark to Light and vice versa

image

image

Verification passed on

Brave 0.64.72 Chromium: 74.0.3729.131 (Official Build) beta(64-bit)
Revision 518a41c1fa7ce1c8bb5e22346e82e42b4d76a96f-refs/branch-heads/3729@{#954}
OS Ubuntu 18.04 LTS

This issue is blocked on Linux, because of #965

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design A design change, especially one which needs input from the design team feature/user-interface All UI related priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes regression release-notes/exclude
Projects
None yet
Development

No branches or pull requests

8 participants