Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Theme color is displayed incorrectly due to transparency #12803

Closed
petemill opened this issue Jan 23, 2018 · 1 comment · Fixed by #12565
Closed

Theme color is displayed incorrectly due to transparency #12803

petemill opened this issue Jan 23, 2018 · 1 comment · Fixed by #12565

Comments

@petemill
Copy link
Member

Description

If a theme color is provided with an alpha transparency channel, we preserve that transparency when displaying the theme color on the tab control background. This then causes the theme color to be blended with the tab bar background color, making the theme color match incorrectly.

Steps to Reproduce

  1. Visit apple.com
  2. Observe tab theme color when tab is active
  3. Observe apple.com's navigation bar color

Actual result:
Colors do not match

Expected result:
Colors match

Reproduces how often:
100%

Brave Version

about:brave info:
0.19 / 0.20

Reproducible on current live release:
👍

@petemill petemill self-assigned this Jan 23, 2018
@petemill petemill mentioned this issue Jan 23, 2018
10 tasks
petemill added a commit that referenced this issue Jan 23, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix #12803
petemill added a commit that referenced this issue Jan 25, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix #12803
@NejcZdovc NejcZdovc added this to the 0.21.x (Beta Channel) milestone Feb 14, 2018
@bsclifton bsclifton modified the milestones: 0.21.x (Beta Channel), 0.22.x (Developer Channel) Feb 20, 2018
petemill added a commit that referenced this issue Feb 20, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix #12803
@bbondy bbondy modified the milestones: 0.22.x (Developer Channel), 0.23.x (Nightly Channel) Feb 25, 2018
petemill added a commit that referenced this issue Mar 3, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix #12803
@alexwykoff alexwykoff added the priority/P4 Minor loss of function. Workaround usually present. label Mar 13, 2018
@alexwykoff alexwykoff modified the milestones: 0.23.x (Nightly Channel), 0.22.x (Developer Channel) Mar 13, 2018
petemill added a commit to petemill/browser-laptop that referenced this issue Mar 15, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix brave#12803
petemill added a commit to petemill/browser-laptop that referenced this issue Mar 16, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix brave#12803

Also do not throw error when parsing color and no valid value is found
petemill added a commit that referenced this issue Mar 16, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix #12803

Also do not throw error when parsing color and no valid value is found
petemill added a commit that referenced this issue Mar 16, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix #12803

Also do not throw error when parsing color and no valid value is found
petemill added a commit that referenced this issue Mar 16, 2018
…the theme color rendering the same as in the content frame.

Blend's the theme color with a white background, which is common in web content.

Fix #12803

Also do not throw error when parsing color and no valid value is found
@LaurenWags
Copy link
Member

LaurenWags commented Mar 22, 2018

Verified on macOS 10.12.6 x64 using the following build:

  • 0.22.6 e6ff4ea
  • libchromiumcontent: 65.0.3325.162
  • muon: 5.1.0

Verified on Windows x64

  • 0.22.6 e6ff4ea
  • libchromiumcontent: 65.0.3325.162
  • muon: 5.1.0

Verified on Ubuntu 10.10 x64

  • 0.22.7 8bb7e77
  • libchromiumcontent: 65.0.3325.181
  • muon: 5.1.1

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

Successfully merging a pull request may close this issue.

8 participants