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

Follow up - Tab separator has too much contrast on both light and dark mode #40714

Closed
1 of 6 tasks
aguscruiz opened this issue Aug 27, 2024 · 11 comments · Fixed by brave/brave-core#25353
Closed
1 of 6 tasks

Comments

@aguscruiz
Copy link

aguscruiz commented Aug 27, 2024

Description

Follow up to #40608

Got feedback that there's still too much contrast between between the separator and the tabbar. Let's do the following:

  • Reduce opacity of separator to 50%
  • Change height of separator from 24px to 16px and center it vertically

That should make it look like the attached screenshot.

Figma reference:
https://www.figma.com/design/H11ZOl6JMYbCTW4ZJXqR5V/%F0%9F%A6%81-Browser?node-id=7913-31400&t=YkcfLB3xQ93kkxjV-1

Steps to reproduce

Actual result

image

Expected result

image

Reproduces how often

Easily reproduced

Brave version (brave://version info)

1.69.155

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

  • with Brave Shields disabled
  • with Brave Rewards disabled
  • in the latest version of Chrome

Miscellaneous information

No response

@aguscruiz
Copy link
Author

@rebron let me know if the expected result screenshot is looking good for you. I think it's subtle enough

@fallaciousreasoning
Copy link

Note: Talked to @aguscruiz and we increased the opacity to 75%

@aguscruiz
Copy link
Author

Latest looks great on this version
https://github.com/brave/brave-browser/releases/tag/v1.71.42

@fallaciousreasoning
Copy link

@LaurenWags should this be marked release-notes/exclude instead?

@LaurenWags
Copy link
Member

LaurenWags commented Aug 28, 2024

@fallaciousreasoning since this issue changes both color and size for tab separator, I marked it as release-notes/include because this change would supersede the changes made with #40608 (which is also release-notes/include for now). definitely open to changing that if others feel differently - was basically just following what we did for 40608.

cc @kjozwiak @rebron for additional input 👍🏻

@kjozwiak
Copy link
Member

The above requires 1.69.159 or higher for 1.69.x verification 👍

@GeetaSarvadnya
Copy link

Verification PASSED on

Brave | 1.69.159 Chromium: 128.0.6613.114 (Official Build) (64-bit)
-- | --
Revision | b60455471a6aad96b2f031c679838da121f22b03
OS | Windows 10 Version 22H2 (Build 19045.4780)
  • Verified the description from the issue on 1.69.159 and ensured the followings:
    • Opacity of the separator reduced to 50%
    • Changed the height of separator from 24px to 16px and center it vertically

1.69.159 - Light theme Vs Dark theme

image
image

1.69.155 - Light theme Vs Dark theme

image
image

1.69.153 - Light theme Vs Dark theme

image
image

@LaurenWags
Copy link
Member

LaurenWags commented Aug 29, 2024

Verified with

Brave | 1.69.159 Chromium: 128.0.6613.114 (Official Build) (x86_64)
-- | --
Revision | b60455471a6aad96b2f031c679838da121f22b03
OS | macOS Version 13.6.9 (Build 22G830)

Compared tab separator colors on 1.69.153 (current release) to 1.69.155 (previous RC with change from #40608) to 1.69.159 for light and dark themes:

1.69.153 1.69.155 1.69.159
153 lt 155 lt 159 lt
153 dk 155 dk 159 dk

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Aug 29, 2024

Verified PASSED on

Brave | 1.69.159 Chromium: 128.0.6613.114 (Official Build) (arm64)
-- | --
Revision | b60455471a6aad96b2f031c679838da121f22b03
OS | macOS Version 12.5 (Build 21G72)

Compared tab separator colors on 1.69.153 (current release) to 1.69.155 (previous RC with change from #40608) to 1.69.159 for light and dark themes:

1.69.153 1.69.155 1.69.159
Screenshot 2024-08-29 at 6 51 28 PM Screenshot 2024-08-29 at 7 03 19 PM Screenshot 2024-08-29 at 7 15 02 PM
Screenshot 2024-08-29 at 6 52 35 PM Screenshot 2024-08-29 at 7 04 25 PM Screenshot 2024-08-29 at 7 15 41 PM

@aguscruiz
Copy link
Author

Looks great on v1.69.159
https://github.com/brave/brave-browser/releases/tag/v1.69.159

image image

@btlechowski
Copy link

Verified with

Brave 1.69.159 Chromium: 128.0.6613.114 (Official Build) (64-bit)
Revision b60455471a6aad96b2f031c679838da121f22b03
OS Linux
image image

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