Skip to content

/ font-kit

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

Use top-left as the origin for raster bounds #113

Merged
merged 1 commit into from Dec 6, 2019
Merged

Use top-left as the origin for raster bounds #113

merged 1 commit into from Dec 6, 2019
+52 −26

Conversation

@jrmuizel
Copy link
Collaborator

jrmuizel commented Nov 24, 2019

This fixes the origin mismatch that happens in the fallback
implementation of raster_bounds() and matches the origin of the canvas
which makes the api more usable.

The major version is bumped because of the behaviour change.
@jrmuizel
Use top-left as the origin for raster bounds
Loading status checks…
2e52c97 
This fixes the origin mismatch that happens in the fallback
implementation of raster_bounds() and matches the origin of the canvas
which makes the api more usable.

The major version is bumped because of the behaviour change.

Fixes #109
@jrmuizel jrmuizel force-pushed the jrmuizel:raster-origin branch from a7f8196 to 2e52c97 Nov 24, 2019
@pcwalton pcwalton merged commit 91a1858 into servo:master Dec 6, 2019
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Aloxaf Aloxaf mentioned this pull request Dec 10, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@jrmuizel @pcwalton
You can’t perform that action at this time.