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

BlendMode::SubpixelWithBgColor text rendering should still be allowed inside transparent groups #2670

Closed
mstange opened this issue Apr 18, 2018 · 1 comment
Assignees
Labels

Comments

@mstange
Copy link
Contributor

@mstange mstange commented Apr 18, 2018

With the default theme on Mac, overflowing tab titles no longer use subpixel AA. However, the text in the tab titles uses a font smoothing background color, and BlendMode::SubpixelWithBgColor is able to render onto transparency without any problems. So I think we need to adjust the logic that disables subpixel AA in this case, so that SubpixelWithBgColor text blending is still allowed.

@kvark kvark added the type: bug label Apr 20, 2018
glennw pushed a commit to glennw/webrender that referenced this issue Apr 27, 2018
When a subpixel text run has a specified background color, we can
safely draw it into an off-screen surface.

Fix a bug in the renderer where the shader color mode was being
set before the shader was bound.

Add support in wrench for specifying the background color of
a text run.

Add a reftest that checks offscreen surface + bg color subpixel
text is the same as subpixel text on a normal background.

Fixes servo#2670.
@glennw glennw self-assigned this Apr 27, 2018
@glennw
Copy link
Member

@glennw glennw commented Apr 27, 2018

Fix is #2701.

glennw pushed a commit to glennw/webrender that referenced this issue Apr 30, 2018
When a subpixel text run has a specified background color, we can
safely draw it into an off-screen surface.

Fix a bug in the renderer where the shader color mode was being
set before the shader was bound.

Add support in wrench for specifying the background color of
a text run.

Add a reftest that checks offscreen surface + bg color subpixel
text is the same as subpixel text on a normal background.

Fixes servo#2670.
bors-servo added a commit that referenced this issue May 1, 2018
Allow subpixel with specified background color in offscreen targets.

When a subpixel text run has a specified background color, we can
safely draw it into an off-screen surface.

Fix a bug in the renderer where the shader color mode was being
set before the shader was bound.

Add support in wrench for specifying the background color of
a text run.

Add a reftest that checks offscreen surface + bg color subpixel
text is the same as subpixel text on a normal background.

Fixes #2670.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2701)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.