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

Add some information about coordinate systems in webrender. #486

Merged
merged 1 commit into from Oct 27, 2016

Conversation

@glennw
Copy link
Member

glennw commented Oct 27, 2016

This is just a start - we need to document each API that
deviates from the conventions listed here, and also investigate
using strongly typed units where it makes sense to.


This change is Reviewable

This is just a start - we need to document each API that
deviates from the conventions listed here, and also investigate
using strongly typed units where it makes sense to.
@glennw
Copy link
Member Author

glennw commented Oct 27, 2016

@mrobinson
Copy link
Member

mrobinson commented Oct 27, 2016

@glennw Looks good to me, but feel free to wait for @pcwalton.

@staktrace
Copy link
Contributor

staktrace commented Oct 27, 2016

Just to confirm - when you say CSS pixels, I'm assuming you mean it doesn't include CSS transforms, and instead those transforms are put into the transform field of the stacking context?

@glennw
Copy link
Member Author

glennw commented Oct 27, 2016

@staktrace Yep, that's right.

@glennw
Copy link
Member Author

glennw commented Oct 27, 2016

@bors-servo r=mrobinson

@bors-servo
Copy link
Contributor

bors-servo commented Oct 27, 2016

📌 Commit 443da92 has been approved by mrobinson

@bors-servo
Copy link
Contributor

bors-servo commented Oct 27, 2016

Testing commit 443da92 with merge 7fa3d80...

bors-servo added a commit that referenced this pull request Oct 27, 2016
Add some information about coordinate systems in webrender.

This is just a start - we need to document each API that
deviates from the conventions listed here, and also investigate
using strongly typed units where it makes sense to.

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

bors-servo commented Oct 27, 2016

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit 443da92 into servo:master Oct 27, 2016
2 of 3 checks passed
2 of 3 checks passed
homu Testing commit 443da92 with merge 7fa3d80...
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@glennw glennw deleted the glennw:some-docs branch Dec 12, 2016
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

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