[Impeller] Attempt to minimize jittering of glyphs when scaling text. #56414
+281
−36
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As I understand, the issue was not caused by the relative positions of glyphs within the run. Instead, by the inconsistent rounding of the scale which was then used to calculate the UV positions.
I have not fully reasoned about the consequences of the fix when the text exceeds the maximum allowable text scale.
This patch does make the linked test case have almost no jitter. The user reported issue also seems to be fixed.
After
Demo.mp4
Before
336405310-be79d790-c570-4590-92a1-4702922517e8.mov