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

Tab Label Rendering Issue #9398

Closed
jonathansampson opened this issue Jun 12, 2017 · 6 comments · Fixed by #10691
Closed

Tab Label Rendering Issue #9398

jonathansampson opened this issue Jun 12, 2017 · 6 comments · Fixed by #10691

Comments

@jonathansampson
Copy link
Collaborator

jonathansampson commented Jun 12, 2017

Test plan

#10691 (comment)


  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Tab labels have an issue with rendering where previous values may not be cleared properly. As a result, this can paint two labels in the place of one.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Desktop (Windows?)

  • Brave Version (revision SHA):
    0.16.9

  • Steps to reproduce:
    TBD

  • Actual result:
    Labels become unreadable.

  • Expected result:
    Labels remain legible and clear.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Screenshot if needed:

image

@jonathansampson jonathansampson added bug design A design change, especially one which needs input from the design team. feature/tabsbar OS/Windows labels Jun 12, 2017
@jonathansampson jonathansampson added this to the 0.19.x milestone Jun 12, 2017
@bsclifton
Copy link
Member

cc: @cezaraugusto @luixxiul

@jonathansampson
Copy link
Collaborator Author

jonathansampson commented Jul 12, 2017

I believe this is related to the general tab-rendering issue where labels are made up of transparent text, masking a gradient background. We need to move to using a pseudo-element to achieve the fade effect. This requires a little work considering we need to accommodate hovered, non-hovered, theme colors, non-theme colors, private tabs, and public tabs.

@bsclifton
Copy link
Member

bsclifton commented Jul 12, 2017

@jonathansampson IIRC, these issues would also be resolved with that same fix:

@jonathansampson
Copy link
Collaborator Author

@bsclifton I believe that is correct.

@cezaraugusto
Copy link
Contributor

@jonathansampson isn't this related to the improvement we talked last Sunday? if so I think you are very close btw.

++ for @bsclifton findings. If above is true I agree they'll be fixed by too.

@jonathansampson
Copy link
Collaborator Author

@cezaraugusto It is. I should have that wrapped up very soon.

@alexwykoff alexwykoff removed this from the 0.19.x (Beta Channel) milestone Jul 18, 2017
@bsclifton bsclifton added usability and removed design A design change, especially one which needs input from the design team. labels Jul 19, 2017
@cezaraugusto cezaraugusto added this to the 0.20.x (Developer Channel) milestone Aug 22, 2017
@cezaraugusto cezaraugusto self-assigned this Aug 22, 2017
cezaraugusto added a commit that referenced this issue Aug 28, 2017
cezaraugusto added a commit that referenced this issue Aug 29, 2017
@ghost ghost added the sprint/1 label Sep 13, 2017
cezaraugusto added a commit that referenced this issue Sep 14, 2017
- text w/ gradient can be buggy. Let's do it with a layer
Auditors: @luixxiul
fix #8414
fix #9398
cezaraugusto added a commit that referenced this issue Sep 14, 2017
- text w/ gradient can be buggy. Let's do it with a layer
Auditors: @luixxiul
fix #8414
fix #9398
cezaraugusto added a commit that referenced this issue Sep 15, 2017
- text w/ gradient can be buggy. Let's do it with a layer
Auditors: @luixxiul
fix #8414
fix #9398
cezaraugusto added a commit that referenced this issue Sep 15, 2017
- text w/ gradient can be buggy. Let's do it with a layer
Auditors: @luixxiul
fix #8414
fix #9398
cezaraugusto added a commit that referenced this issue Sep 19, 2017
- text w/ gradient can be buggy. Let's do it with a layer
Auditors: @luixxiul
fix #8414
fix #9398
cezaraugusto added a commit that referenced this issue Sep 21, 2017
- text w/ gradient can be buggy. Let's do it with a layer
Auditors: @luixxiul
fix #8414
fix #9398
syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
- text w/ gradient can be buggy. Let's do it with a layer
Auditors: @luixxiul
fix brave#8414
fix brave#9398
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.