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

frame_builder: Don't use subpixel AA for non-opaque text. #1423

Merged
merged 1 commit into from Jun 23, 2017

Conversation

@emilio
Copy link
Member

emilio commented Jun 22, 2017

This fixes servo/servo#17464.

Not 100% sure it's the correct fix, but it seemed reasonable given we disable it
for non-opaque stacking contexts.


This change is Reviewable

This fixes servo/servo#17464.

Not 100% sure it's the correct fix, but it seemed reasonable given we disable it
for non-opaque stacking contexts.
@emilio
Copy link
Member Author

emilio commented Jun 22, 2017

r? @glennw or @kvark

@glennw
Copy link
Member

glennw commented Jun 22, 2017

@emilio Yup, it's certainly an improvement over what we have - we can look at a "proper" solution later. Thanks!

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Jun 22, 2017

📌 Commit 61c70ca has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Jun 23, 2017

Testing commit 61c70ca with merge 1291c3c...

bors-servo added a commit that referenced this pull request Jun 23, 2017
frame_builder: Don't use subpixel AA for non-opaque text.

This fixes servo/servo#17464.

Not 100% sure it's the correct fix, but it seemed reasonable given we disable it
for non-opaque stacking contexts.

<!-- 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/1423)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jun 23, 2017

☀️ Test successful - status-travis
Approved by: glennw
Pushing 1291c3c to master...

@bors-servo bors-servo merged commit 61c70ca into master Jun 23, 2017
4 of 6 checks passed
4 of 6 checks passed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
dependency-ci Dependencies checked
Details
homu Test successful
Details
@emilio emilio deleted the no-subpixel-aa-non-opaque-text branch Jun 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

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