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

apply subpixel-aa to non-blurred text-shadows #1483

Closed
Gankra opened this issue Jul 14, 2017 · 0 comments
Closed

apply subpixel-aa to non-blurred text-shadows #1483

Gankra opened this issue Jul 14, 2017 · 0 comments

Comments

@Gankra
Copy link
Contributor

@Gankra Gankra commented Jul 14, 2017

Gecko does this. Should hopefully be a simple fast-path.

glennw pushed a commit to glennw/webrender that referenced this issue Jul 17, 2017
* Clamp UV fetches to 0.5 texels inside the UV rect for ps_cache and cs_blur shaders.
  Hopefully, this will fix the text-shadow artifacts that appear on some machines / tests.
* Add fast path for text that has blur radius of zero.
  This is a useful optimization, and also needed to pass some of the Servo reftests, which don't support fuzzy tests.
  Text runs with a zero shadow get pushed through the normal text run path, which now supports an offset parameter.
* Respect subpixel AA mode for text runs with zero blur radius.
  Fixes servo#1483.
* Respect disabled text AA mode for text runs with zero blur radius.
  This is needed by some of the Servo reftests.
* Simplify the GPU cache layout of text-shadow primitives.
  They now match that of a normal text run primitive, but have several runs concatenated.
bors-servo added a commit that referenced this issue Jul 17, 2017
Various improvements to text-shadow support.

* Clamp UV fetches to 0.5 texels inside the UV rect for ps_cache and cs_blur shaders.
  Hopefully, this will fix the text-shadow artifacts that appear on some machines / tests.
* Add fast path for text that has blur radius of zero.
  This is a useful optimization, and also needed to pass some of the Servo reftests, which don't support fuzzy tests.
  Text runs with a zero shadow get pushed through the normal text run path, which now supports an offset parameter.
* Respect subpixel AA mode for text runs with zero blur radius.
  Fixes #1483.
* Respect disabled text AA mode for text runs with zero blur radius.
  This is needed by some of the Servo reftests.
* Simplify the GPU cache layout of text-shadow primitives.
  They now match that of a normal text run primitive, but have several runs concatenated.

<!-- 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/1488)
<!-- 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.

1 participant
You can’t perform that action at this time.