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

Native buttons should use brave brand style #6398

Closed
petemill opened this issue Oct 10, 2019 · 5 comments · Fixed by brave/brave-core#3569
Closed

Native buttons should use brave brand style #6398

petemill opened this issue Oct 10, 2019 · 5 comments · Fixed by brave/brave-core#3569
Assignees
Labels
design A design change, especially one which needs input from the design team priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/include

Comments

@petemill
Copy link
Member

petemill commented Oct 10, 2019

Test plan

See brave/brave-core#3569

Original issue description

Infobars and bubble dialogs are still using chromium button layout style and colors.

Description

Design spec has been evolving and still not locked down, but can be interpreted enough in order to conform to chromium button types.

Steps to Reproduce

Infobars

  1. Fresh profile - view the infobars for Analytics and Default Browser

Dialogs
2. Visit maps.google.com and click the (+) icon in the Locaiton Bar

More specific Test plan with all the states expected on brave/brave-core#3569

Actual result:

image

Expected result:

image

Addresses umbrella at #3961

@petemill petemill added design A design change, especially one which needs input from the design team priority/P3 The next thing for us to work on. It'll ride the trains. labels Oct 10, 2019
@petemill petemill self-assigned this Oct 10, 2019
@bsclifton
Copy link
Member

bsclifton commented Oct 10, 2019

@petemill I get a different expected result - not sure how you got Install on the left?
(it's a nitpick, but wanted to make sure I'm not missing something 😛)
Screen Shot 2019-10-10 at 1 55 16 PM

@petemill
Copy link
Member Author

@bsclifton Windows has the primary and secondary actions in dialog reversed. My screenshots were from a mix of macOS and Windows. Just to keep your mind active!

@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

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

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 1, 2019

Verification is 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)

  • Regular states for different buttons in Dark mode
    Settings page buttons
    image
    image
    image
    Bookmraks buttons in toolbar
    image
    image
    image
    Welcome screen buttons
    image
    image
    image
    image
    image
    image
    Edit bookmarks buttons in toolbar
    image
    image
    image
    chrome://settings/importData buttons
    image
    image
    image
    Crypo wallet
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    Verified one backup phrase out of 24 all buttons behaves the same way
    image
    image
    image
    chrome://bookmarks/ -> Add a new folder or Add a new bookmark or Edit bookmarks buttons
    image
    image
    Rewards
    image
    image

  • Hover states for different buttons in Dark mode
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image

  • Pressed states for different buttons in Dark mode
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image

Verified passed with

Brave 0.71.112 Chromium: 78.0.3904.87 (Official Build) beta (64-bit)
Revision 20c21f4010010f32462ea8e1d6af30cef66d48c8-refs/branch-heads/3904@{#840}
OS macOS Version 10.13.6 (Build 17G5019)

Native UI buttons example - Light theme
Display
Hover - Prominent:
Hover-Prominent
Hover-Regular:
Hover-Regular
Click-Prominent:
click-Prominent
Click-Regular:
click-Regular

WebUI buttons example - Light theme
Display2
Hover-Prominent:
Hover-Prominent2
Hover-Regular:
Hover-Regular2
Click-Prominent:
Click-Prominent2
Click-Regular:
Click-Regular2

Native UI buttons example - Dark theme
Display-Dk
Hover-Prominent:
Hover-Prominent-Dk
Hover-Regular:
Hover-Regular-Dk
Click-Prominent:
Click-Prominent-Dk
Click-Regular:
Click-Regular-Dk

WebUI buttons example - Dark theme
Display-Dk2
Hover-Prominent:
Hover-Prominent-Dk2
Hover-Regular:
Hover-Regular-Dk2
Click-Prominent:
Click-Prominent-Dk2
Click-Regular:
Click-Regular-Dk2

Encountered #6770 for Crypto Wallets buttons and #6783 for button on welcome flow
Reproduced various issues noted in #6707

Spot checked buttons in the following areas:

  • brave://settings/getStarted
  • brave://settings/searchEngines
  • brave://settings/content (spot checked various sections within this area)
  • brave://settings/clearBrowserData
  • brave://settings/payments
  • brave://settings/addresses
  • brave://settings/downloads
  • brave://settings/reset
  • brave://history (deleting an item)
  • brave://bookmarks (editing an item)
  • brave://welcome
  • creating a bookmark from URL bar
  • button in ‘Default’ banner notification on relaunch of Brave
  • download bar

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

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 priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants