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

Update theme colors for light, dark, private window and private window with Tor #31644

Closed
rebron opened this issue Jul 14, 2023 · 13 comments · Fixed by brave/brave-core#20535
Closed

Comments

@rebron
Copy link
Collaborator

rebron commented Jul 14, 2023

Description

Update the theme colors for light, dark, private window and private window with Tor.

Design

Screenshot 2023-07-14 at 8 23 52 AM

Figma: https://www.figma.com/file/H11ZOl6JMYbCTW4ZJXqR5V/%F0%9F%A6%81-Browser?type=design&node-id=3014%3A33123&mode=design&t=nhfxV22NiGDSYN2e-1

Reproduces how often:

n/a

Brave version (brave://version info)

n/a

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? n/a
  • Can you reproduce this issue with the nightly channel? n/a

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:

simonhong added a commit to brave/brave-core that referenced this issue Nov 1, 2023
@brave-builds brave-builds added this to the 1.62.x - Nightly milestone Nov 1, 2023
@MadhaviSeelam
Copy link

MadhaviSeelam commented Nov 7, 2023

Verification PASSED using

Brave | 1.62.28 Chromium: 119.0.6045.105 (Official Build) nightly (64-bit)
-- | --
Revision | af1dadeae432bc6f1bed980ac4123d5b7dda6e24
OS | Windows 11 Version 22H2 (Build 22621.2428)
  • Installed 1.62.28
  • launched Brave
  • imported bookmarks
  • opened few tabs, pinned tabs

Normal window

Light theme - horizontal tab view

1.60.x 1.62.x
image image

Light theme - vertical tab view

1.60.x 1.62.x
image image

Dark theme - horizontal tab view

1.60.x 1.62.x
image image

Dark theme - vertical tab view

1.60.x 1.62.x
image image

Private window

Light theme - horizontal tab view

1.60.x 1.62.x
image image

Light theme - vertical tab view

1.60.x 1.62.x
image image

Dark theme - horizontal tab view

1.60.x 1.62.x
image image

Dark theme - vertical tab view

1.60.x 1.62.x
image image

Private window with Tor

Light theme - horizontal tab view 1.60.x | 1.62.x ----- | ----- image|image

Light theme - vertical tab view

1.60.x 1.62.x
image image

Dark theme - horizontal tab view

1.60.x 1.62.x
image image

Dark theme - vertical tab view

1.60.x 1.62.x
image image

Guest window

1.60.x 1.62.x
image image

Profile custom theme

Light theme - horizontal Light theme - vertical
image image
Dark theme - horizontal Dark theme - vertical
image image
Light theme - horizontal Light theme - vertical
image image
Dark theme - horizontal Dark theme - vertical
image image

@aguscruiz
Copy link

Just noting here that in private/tor windows, we're showing a dark mode header and light mode content.
We should show them according to the mode the user selected and not mix them

image

@MadhaviSeelam MadhaviSeelam added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Nov 7, 2023
@simonhong
Copy link
Member

Just noting here that in private/tor windows, we're showing a dark mode header and light mode content. We should show them according to the mode the user selected and not mix them

image

Filed #34165

mherrmann pushed a commit to brave/brave-core that referenced this issue Nov 9, 2023
@stephendonner stephendonner added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Dec 11, 2023
@stephendonner
Copy link

stephendonner commented Dec 11, 2023

Verification PASSED using

Brave | 1.62.104 Chromium: 120.0.6099.71 (Official Build) beta (x86_64)
-- | --
Revision | e2b266991247b909fa88520bb12d018223872d29
OS | macOS Version 11.7.10 (Build 20G1427)

Normal window

Light theme - horizontal tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 4 48 59 PM Screenshot 2023-12-11 at 5 00 16 PM

Light theme - vertical tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 4 49 16 PM Screenshot 2023-12-11 at 5 00 09 PM

Dark theme - horizontal tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 4 50 33 PM Screenshot 2023-12-11 at 4 59 35 PM

Dark theme - vertical tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 4 50 38 PM Screenshot 2023-12-11 at 4 59 41 PM

Private window

Light theme - horizontal tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 3 25 05 PM Screenshot 2023-12-11 at 3 42 27 PM

Light theme - vertical tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 3 57 42 PM Screenshot 2023-12-11 at 3 52 03 PM

Dark theme - horizontal tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 3 40 03 PM Screenshot 2023-12-11 at 3 42 37 PM

Dark theme - vertical tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 4 04 03 PM Screenshot 2023-12-11 at 3 51 54 PM

Private window with Tor

Light theme - horizontal tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 1 54 10 PM Screenshot 2023-12-11 at 1 46 57 PM

Light theme - vertical tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 4 16 04 PM Screenshot 2023-12-11 at 4 21 12 PM

Dark theme - horizontal tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 1 55 58 PM Screenshot 2023-12-11 at 1 47 15 PM

Dark theme - vertical tab view

1.61.x 1.62.x
Screenshot 2023-12-11 at 4 16 15 PM Screenshot 2023-12-11 at 4 25 13 PM

Guest window

1.61.x 1.62.x
Screenshot 2023-12-11 at 2 02 17 PM Screenshot 2023-12-11 at 2 05 16 PM

Custom-theme profile

Profile 1, Dark Teal

Light theme - horizontal Light theme - vertical
Screen Shot 2023-12-11 at 6 43 47 PM Screen Shot 2023-12-11 at 6 44 07 PM
Dark theme - horizontal Dark theme - vertical
Screen Shot 2023-12-11 at 6 47 05 PM Screen Shot 2023-12-11 at 6 47 10 PM

Profile 2, Beige and White

Light theme - horizontal Light theme - vertical
Screen Shot 2023-12-11 at 6 49 59 PM Screen Shot 2023-12-11 at 6 50 04 PM
Dark theme - horizontal Dark theme - vertical
Screen Shot 2023-12-11 at 6 49 34 PM Screen Shot 2023-12-11 at 6 51 25 PM

@stephendonner stephendonner added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Dec 12, 2023
@btlechowski
Copy link

Verification PASSED using

Brave 1.62.122 Chromium: 120.0.6099.144 (Official Build) beta (64-bit)
Revision cfddebe77d394064c472fda64afcd9fbed34ceb4
OS Linux

Normal window

Light theme - horizontal tab view

image

Light theme - vertical tab view

image

Dark theme - horizontal tab view

image

Dark theme - vertical tab view

image

Private window

Light theme - horizontal tab view

image

Light theme - vertical tab view

image

Dark theme - horizontal tab view

image

Dark theme - vertical tab view

image

Private window with Tor

Light theme - horizontal tab view

image

Light theme - vertical tab view

image

Dark theme - horizontal tab view

image

Dark theme - vertical tab view

image

Guest window

image

Custom-theme profile

Profile 1, Dark Teal

Light theme - horizontal Light theme - vertical
image image
Dark theme - horizontal Dark theme - vertical
image image

@nintendork07
Copy link

nintendork07 commented Jan 25, 2024

any chance this change could get reversed??? the new themes are too dark and text is much less legible, especially in private windows. the icons are also basically invisible. it's giving me a lot more eye strain trying to read the tab & URL text fields

@mc0ps
Copy link

mc0ps commented Jan 28, 2024

I loved the default theme color for dark mode before this change. Not sure what the motivation was to mess with something that everyone seemed to like. I have a similar issue with the recent "polishing" of the icons by rounding the corners. Please consider reverting these styling changes or give users an option to retain the legacy styling.

@drkdw
Copy link

drkdw commented Jan 29, 2024

I like the new dark theme!

I do have a question, how do you place the slack, twitter and gmail icon next to the tabs? I want to try that too.

Edit: I just found it, you have to 'pin' the tab. I do notice my Brave doesn't have the same look & feel for these icons. I don't have that border around the icons and not so well aligned; see screenshot:
SCR-20240129-kqit

@JoshAntBrown
Copy link

I'm all for improvements, but I don't understand how this can be considered an improvement. I'm sure dark mode probably looks nice, but the light mode theme now looks washed out and broken.

I've left some feedback on the community forum: https://community.brave.com/t/what-did-you-do-the-ui-looks-broken/528747.

If you want to introduce a new theme for some reason, then build it as a theme and make it the default, allow people to revert back to the previous theme.

@aguscruiz
Copy link

Hi everyone, thanks for the feedback.
We're working on some color tweaks that should improve contrast and brightness a bit.

@drkdw if you want to test out the new look and feel for the icons, you have to enable this flag #brave-horizontal-tabs-update. We're testing it out internally first :)

@drkdw
Copy link

drkdw commented Jan 30, 2024

@aguscruiz Thx! Looks great ;)

@stephendonner
Copy link

We've landed a followup to lighten-up some of these colors; you can see it using the current nightly builds: #35624

@nintendork07
Copy link

was this changed again? i updated to 1.70.117 today and the colors are too way dark.

Screenshot 2024-09-19 at 3 10 23 PM

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

Successfully merging a pull request may close this issue.