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

[Nala]: Remove Chromium color overrides #40689

Closed
fallaciousreasoning opened this issue Aug 27, 2024 · 3 comments · Fixed by brave/brave-core#25337
Closed

[Nala]: Remove Chromium color overrides #40689

fallaciousreasoning opened this issue Aug 27, 2024 · 3 comments · Fixed by brave/brave-core#25337

Comments

@fallaciousreasoning
Copy link

fallaciousreasoning commented Aug 27, 2024

Description

With the new Nala theming work @aguscruiz has been doing, it should be possible for us to remove a bunch of the overriding logic we have in our brave_color_mixer and brave_tabs_color_mixer and just use the underlying colors from Chromium.

As an added bonus, this means we can enable the tri-color themes Chromium is using now before they remove support for the old dual color themes.

*Note:
As theming changes (especially to dark mode) are often controversial we should have a blog post explaining the changes.

Solid Color Themes

Existing users, who've picked one of the solid color themes will not be affected by these changes. However, if they change their theme color the only way to get the solid color theme back will be to install that theme from the Chrome Web Store.

Default Theme for New Users

By default Chromium sets the default theme to be the Gray/Blue theme. This is a bit different to the current default theme in Brave which is Gray/light gray. We should ensure that we default to the Gray/Gray theme.

Dark Theme Changes

This is likely to be the most controversial part of the change. The new dark theme is slightly lighter than the existing dark theme, and is inherited from upstream. This will make it significantly easier for us to maintain, and there will be far fewer override points after this change.

However, people are often pretty unhappy when the colors in their browser change. As a mitigation, in a blog post we should point users to the Just Black theme in the Chrome Web Store, which should result in a suitably dark theme.

cc @rebron & @aguscruiz for all the stuff I've forgotten to mention.

Approach

There are few parts to this change:

  1. Remove unnecessary overrides
  2. Enable Tri-Color themes by reverting patches
  3. Ensure Gray/Gray theme is selected by default
@MadhaviSeelam
Copy link

MadhaviSeelam commented Sep 24, 2024

Verification PASSED using

Brave | 1.71.94 Chromium: 129.0.6668.59 (Official Build) beta (64-bit)
-- | --
Revision | 90ea886c66dbbc8d7c5aecda57b6ef3c4d513497
OS | Windows 11 Version 23H2 (Build 22631.4169)
  1. Installed 1.71.94
  2. opened brave://settings/manageProfile in a new tab
  3. confirmed new tri-color theme is shown with Grey is selected as a default color
  4. confirmed the new themes have a light/dark mode (which are different, unlike the old dual color themes)
  5. The new light/dark theme should be largely similar to the existing light/dark theme????
  6. confirmed vertical tabs looked as expected in light/dark mode
  7. confirmed horizontal tabs looked as expected in light/dark mode
  8. confirmed Omnibox still have hover/selected state when open
  9. confirmed If texted is edited in the omnibox and close it, the omnibox should look the same as when its just showing the URL
Case 1: Normal Window - Default (Gray/Gray) - PASSED
Horizontal tab view
  • Confirmed tab bar/inactive tabs in horizontal view shown in default gray color
  • Confirmed active tab shown in white
1.70.x 1.71.x
image 2024-09-23_13h28_50
2024-09-23_13h37_54 image
image image
image image
Vertical tabs - PASSED
  • Confirmed vertical tab strip is not dark compared to 1.70.x which is expected as per the issue description
  • Confirmed active tab shown darker than 1.70.x
1.70.x 1.71.x
image image
image image
omnibox hover/selected state when open - PASSED
  • Entered a string in the omnibox and hovered over the entries

  • clicked in the omnibox

  • opened a new tab and click anywhere to show placeholder in the omnibox

  • Confirmed when hovered over, result entries were highlighted with gray

  • Confirmed when hovered over on selected result, result was highlighted with gray

  • Confirmed empty omnibox has reddish tone of Gray
     
    1.70.x | 1.71.x
    ----- | -----
    image|image
    image|2024-09-23_14h25_38
    image|image

  • Background text and font of the results have coordinated colors in the omnibox

example example example
image image image
  • confirmed If texted is edited in the omnibox and close it, the omnibox should look the same as when its just showing the URL
example example example
image image image
Case 2: New Window - Violet - PASSED
  • continued from Case 1 above
  • opened New window via hamburger menu
  • opened brave://settings/manageProfile
  • selected Violet theme

Horizontal tab view

example example example example
image image image image

Vertical tab view

example example example example
image image image image
Multiple profiles - system generated custom color - PASSED
  • Add new profile via hamburger menu
  • verified custom color was picked by default
  • entered a new profile name (Profile 2)
  • clicked Done
example example example example example
image image image image image
Install dark theme from Chrome store - PASSED
  • Installed 1.71.94
  • install Just Black theme via chrome webstore

Horizontal tab view and vertical tab view looks a little darker with Just Black than Brave dark theme

example example example example
image image image image
Upgrade - PASSED
Existing theme is retained - Light green to Orange - PASSED
  1. Installed 1.70.117
  2. launched Brave
  3. opened brave://settings/manageProfile
  4. selected Light green
  5. closed Brave
  6. renamed the profile Brave-Browser-Beta
  7. installed 1.71.94
  8. launched Brave
  • Confirmed Light green theme is retained
  • Confirmed tri color themes is shown in the brave://settings/manageProfile
  • Confirmed Light green is populated in the Custom color widget
  • Confirmed omnibox with a URL placeholder text shows whitish
  • Confirmed when hovered over, result entries were highlighted with gray
  • Confirmed when hovered over on selected result, result was highlighted with gray
step 1 step 3 step 7 step 10 -11 example example example example
image image image image image image image image
Change from Dark purple to Aqua to Ultra Violet - PASSED
  1. Installed 1.70.117
  2. launched Brave
  3. opened brave://settings/manageProfile
  4. selected Dark purple
  5. closed Brave
  6. renamed the profile Brave-Browser-Beta
  7. installed 1.71.94
  8. launched Brave
  9. opened brave://settings/manageProfile
  10. switched theme from Aqua
  11. clicked on Custom color widget which has green to switch back to previous theme
  12. Color picker popup is shown
  13. loaded https://chromewebstore.google.com/collection/chrome_color_themes
  14. selected Ultra Violet theme
  15. clicked Add to Brave
  • Confirmed Dark purple theme is retained
  • Confirmed tri color themes is shown in the brave://settings/manageProfile
  • Confirmed Dark purple is populated in the Custom color widget
  • Confirmed theme was changed from to Aqua
    -Confirmed unable to switch it back to previous theme Light green instead Color picker is shown
    -Confirmed Ultra Violet theme was added which was close to Dark purple
step 4 step 9 step 10 step 12 step 13 step 14 step 15
image image image image image image image

@MadhaviSeelam MadhaviSeelam added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Sep 25, 2024
@MadhaviSeelam MadhaviSeelam added QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Oct 3, 2024
@LaurenWags LaurenWags added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Oct 7, 2024
@LaurenWags
Copy link
Member

LaurenWags commented Oct 7, 2024

Verified with

Brave | 1.71.105 Chromium: 129.0.6668.89 (Official Build) beta (x86_64)
-- | --
Revision | f69ba167992437799f1dbcce81e90ba501a3d63e
OS | macOS Version 14.7 (Build 23H124)
  1. Installed 1.71.x
  2. opened brave://settings/manageProfile in a new tab
  3. confirmed new tri-color theme is shown with Grey is selected as a default color
Light theme Dark theme
Screenshot 2024-10-07 at 9 49 04 AM Screenshot 2024-10-07 at 9 49 13 AM
Screenshot 2024-10-07 at 9 52 48 AM Screenshot 2024-10-07 at 9 53 13 AM
  1. confirmed the new themes have a light/dark mode (which are different, unlike the old dual color themes)
Light theme Dark theme
Screenshot 2024-10-07 at 9 50 22 AM Screenshot 2024-10-07 at 9 50 33 AM
Screenshot 2024-10-07 at 9 54 27 AM Screenshot 2024-10-07 at 9 58 51 AM
Screenshot 2024-10-07 at 9 54 35 AM Screenshot 2024-10-07 at 9 54 55 AM
  1. The new light/dark theme should be largely similar to the existing light/dark theme

Note

There is no separator (most visible on light theme) on 1.71.x due to rounded corners which is currently only enabled on Nightly and Beta. Fix coming for this with brave/brave-core#25738.

1.70.x 1.71.x
Screenshot 2024-10-07 at 10 09 13 AM Screenshot 2024-10-07 at 10 08 55 AM
Screenshot 2024-10-07 at 10 11 43 AM Screenshot 2024-10-07 at 10 11 50 AM
Screenshot 2024-10-07 at 10 19 37 AM Screenshot 2024-10-07 at 10 19 46 AM
Screenshot 2024-10-07 at 10 18 01 AM Screenshot 2024-10-07 at 10 18 07 AM
  1. confirmed vertical tabs looked as expected in light/dark mode

Note

Encountered #41450 which exists in all channels, not a regression from this PR.

Light theme Dark theme
Screenshot 2024-10-07 at 10 33 01 AM Screenshot 2024-10-07 at 10 33 19 AM
  1. confirmed horizontal tabs looked as expected in light/dark mode
Light theme Dark theme
Screenshot 2024-10-07 at 10 26 17 AM Screenshot 2024-10-07 at 10 27 17 AM
Screenshot 2024-10-07 at 10 27 58 AM Screenshot 2024-10-07 at 10 28 15 AM
  1. confirmed Omnibox still have hover/selected state when open
Light theme Dark theme
Screenshot 2024-10-07 at 10 49 22 AM Screenshot 2024-10-07 at 10 48 14 AM
Screenshot 2024-10-07 at 10 49 30 AM Screenshot 2024-10-07 at 10 48 27 AM
Screenshot 2024-10-07 at 10 49 42 AM Screenshot 2024-10-07 at 10 49 52 AM
Screenshot 2024-10-07 at 10 50 41 AM Screenshot 2024-10-07 at 10 50 29 AM
  1. confirmed If texted is edited in the omnibox and close it, the omnibox should look the same as when its just showing the URL
. Light theme Dark theme
Omnibox not in focus (no text entered) Screenshot 2024-10-07 at 1 54 09 PM Screenshot 2024-10-07 at 1 55 20 PM
Omnibox in focus with text entered (showing drop down) Screenshot 2024-10-07 at 1 54 18 PM Screenshot 2024-10-07 at 1 55 29 PM
Omnibox in focus with text (drop down not shown) Screenshot 2024-10-07 at 1 54 33 PM Screenshot 2024-10-07 at 1 55 36 PM
Omnibox with text entered, not in focus Screenshot 2024-10-07 at 1 54 39 PM Screenshot 2024-10-07 at 1 55 43 PM
Case 1 - default light/dark theme - PASSED

See above for screenshots.
Covered horizontal and vertical tab views as well as light and dark themes.

Case 2: Theme color with light/dark - PASSED
  • opened New window via hamburger menu
  • opened brave://settings/manageProfile
  • selected "Violet" theme
  • Compared light/dark tinting to "Violet" theme for both horizontal and vertical tab views
Light Dark
Screenshot 2024-10-07 at 2 24 00 PM Screenshot 2024-10-07 at 2 31 40 PM
Screenshot 2024-10-07 at 2 24 08 PM Screenshot 2024-10-07 at 2 26 31 PM
Screenshot 2024-10-07 at 2 24 50 PM Screenshot 2024-10-07 at 2 26 50 PM
Screenshot 2024-10-07 at 2 25 02 PM Screenshot 2024-10-07 at 2 27 06 PM
Screenshot 2024-10-07 at 2 25 12 PM Screenshot 2024-10-07 at 2 27 16 PM
Multiple profiles - system generated custom color - PASSED
  • Add new profile via hamburger menu
  • verified custom color was picked by default
  • entered a new profile name (Profile 2)
  • clicked Done
example example example example example
Screenshot 2024-10-07 at 2 54 57 PM Screenshot 2024-10-07 at 2 55 08 PM Screenshot 2024-10-07 at 2 55 27 PM Screenshot 2024-10-07 at 2 56 03 PM Screenshot 2024-10-07 at 2 56 28 PM
Install dark theme from Chrome store - PASSED
  • Installed 1.71.x
  • install Morpheon Dark theme via chrome webstore

Horizontal tab view and vertical tab view looks darker with Morpheon Dark than Brave dark theme

example example example
Screenshot 2024-10-07 at 3 09 13 PM Screenshot 2024-10-07 at 3 09 22 PM Screenshot 2024-10-07 at 3 09 40 PM
Upgrade - PASSED
Existing theme is retained - PASSED
  1. Installed 1.70.123
  2. launched Brave
  3. opened brave://settings/manageProfile
  4. selected purple
Example Example
Screenshot 2024-10-07 at 3 13 01 PM Screenshot 2024-10-07 at 3 14 07 PM
  1. closed Brave
  2. renamed the profile Brave-Browser-Beta
  3. installed 1.71.105
  4. launched Brave
  • Confirmed bright purple theme is retained
  • Confirmed tri color themes is shown in the brave://settings/manageProfile
  • Confirmed bright purple is populated in the Custom color widget
  • Confirmed omnibox with a URL placeholder text shows white-ish
  • Confirmed when hovered over, result entries were highlighted with gray
  • Confirmed when hovered over on selected result, result was highlighted with gray
example example example example example
Screenshot 2024-10-07 at 3 15 14 PM Screenshot 2024-10-07 at 3 15 02 PM Screenshot 2024-10-07 at 3 15 43 PM Screenshot 2024-10-07 at 3 15 59 PM Screenshot 2024-10-07 at 3 16 23 PM
Change from purple to Aqua to Ultra Violet - PASSED
  1. Installed 1.70.123
  2. launched Brave
  3. opened brave://settings/manageProfile
  4. selected purple
Screenshot 2024-10-07 at 3 40 49 PM
  1. closed Brave
  2. renamed the profile Brave-Browser-Beta
  3. installed 1.71.105
  4. launched Brave
  5. opened brave://settings/manageProfile
  6. confirmed theme is retained (as custom color)
Screenshot 2024-10-07 at 3 41 23 PM
  1. switched theme to one of the pre-set tri-color ones: Aqua
Screenshot 2024-10-07 at 3 41 50 PM
  1. clicked on Custom color widget
  2. Color picker popup is shown (did not select a custom color, left profile color as Aqua)
Screenshot 2024-10-07 at 3 42 09 PM
  1. loaded https://chromewebstore.google.com/collection/chrome_color_themes
  2. selected Ultra Violet theme
  3. clicked Add to Brave
Screenshot 2024-10-07 at 3 43 00 PM

@LaurenWags LaurenWags added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Oct 7, 2024
@btlechowski
Copy link

Verification PASSED using

Brave 1.71.113 Chromium: 130.0.6723.58 (Official Build) (64-bit)
Revision 3ee74228044546a84c661332ddb441a7a68108fa
OS Linux
  1. Installed 1.71.113
  2. opened brave://settings/manageProfile in a new tab
  3. confirmed new tri-color theme is shown with Grey is selected as a default color
  4. confirmed the new themes have a light/dark mode (which are different, unlike the old dual color themes)
  5. The new light/dark theme should be largely similar to the existing light/dark theme????
  6. confirmed vertical tabs looked as expected in light/dark mode
  7. confirmed horizontal tabs looked as expected in light/dark mode
  8. confirmed Omnibox still have hover/selected state when open
  9. confirmed If texted is edited in the omnibox and close it, the omnibox should look the same as when its just showing the URL
Case 1: Normal Window - Default (Gray/Gray) - PASSED
Horizontal tab view
  • Confirmed tab bar/inactive tabs in horizontal view shown in default gray color
  • Confirmed active tab shown in white
Image Image Image Image
Image Image Image Image
Vertical tabs - PASSED
  • Confirmed vertical tab strip is not dark compared to 1.70.x which is expected as per the issue description
  • Confirmed active tab shown darker than 1.70.x
Image Image
omnibox hover/selected state when open - PASSED
  • Entered a string in the omnibox and hovered over the entries

  • clicked in the omnibox

  • opened a new tab and click anywhere to show placeholder in the omnibox

  • Confirmed when hovered over, result entries were highlighted with gray

  • Confirmed when hovered over on selected result, result was highlighted with gray

  • Confirmed empty omnibox has reddish tone of Gray
     

Image Image
Image Image
  • Background text and font of the results have coordinated colors in the omnibox
Image Image
Case 2: New Window - PASSED
  • continued from Case 1 above
  • opened New window via hamburger menu
  • opened brave://settings/manageProfile
  • selected Violet theme

Horizontal tab view

Image Image Image Image

Vertical tab view

Image|Image|Image
--|--

Multiple profiles - system generated custom color - PASSED
  • Add new profile via hamburger menu
  • verified custom color was picked by default
  • entered a new profile name (Profile 2)
  • clicked Done
Image Image Image Image
Install dark theme from Chrome store - PASSED
  • Installed 1.71.94
  • install Just Black theme via chrome webstore

Horizontal tab view and vertical tab view looks a little darker with Just Black than Brave dark theme

Image Image
Upgrade - PASSED
Existing theme is retained - PASSED
  1. Installed 1.70.x
  2. launched Brave
  3. opened brave://settings/manageProfile
  4. selected Pink
  5. closed Brave
  6. installed 1.71.x
  7. launched Brave
  • Confirmed tri color themes is shown in the brave://settings/manageProfile
  • Confirmed omnibox with a URL placeholder text shows whitish
  • Confirmed when hovered over, result entries were highlighted with gray
  • Confirmed when hovered over on selected result, result was highlighted with gray

1.70.x

Image Image

1.71.x
Image

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