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

Sync changes from mozilla-central #3808

Merged
merged 1 commit into from Dec 1, 2019
Merged

Sync changes from mozilla-central #3808

merged 1 commit into from Dec 1, 2019

Conversation

@moz-gfx
Copy link

moz-gfx commented Dec 1, 2019

No description provided.

Snapping glyph positions are an internal detail to a primitive. As such,
any snapping required must be taken into account when calculating the
local rect. That ensures that when the clip is applied, it doesn't cut
off parts of the glyph that would have been retained after snapping.

Differential Revision: https://phabricator.services.mozilla.com/D55348

[wrupdater] From https://hg.mozilla.org/mozilla-central/rev/b5cfc7ed22b48babd34b6f1cbd2b1b7d93648a07
@moz-gfx
Copy link
Author

moz-gfx commented Dec 1, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Dec 1, 2019

📌 Commit 1ab7837 has been approved by moz-gfx

@bors-servo
Copy link
Contributor

bors-servo commented Dec 1, 2019

Testing commit 1ab7837 with merge 70220dc...

bors-servo added a commit that referenced this pull request Dec 1, 2019
Sync changes from mozilla-central
@bors-servo
Copy link
Contributor

bors-servo commented Dec 1, 2019

☀️ Test successful - status-appveyor, status-taskcluster
Approved by: moz-gfx
Pushing 70220dc to master...

@bors-servo bors-servo merged commit 1ab7837 into servo:master Dec 1, 2019
3 checks passed
3 checks passed
Community-TC (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
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.

None yet

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