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

Fix pre_scale on transform2d and transform3d #377

Merged
merged 1 commit into from Nov 25, 2019

Conversation

@nickwilcox
Copy link
Contributor

nickwilcox commented Nov 23, 2019

Fix the pre_scale method on both Transform2D and Transform3D plus added a test to make sure pre_scale and post_scale matched the equivalent sequence using full transforms.

@nical
Copy link
Collaborator

nical commented Nov 25, 2019

Oops! Thanks!
@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Nov 25, 2019

📌 Commit b267280 has been approved by nical

@bors-servo
Copy link
Contributor

bors-servo commented Nov 25, 2019

Testing commit b267280 with merge 759cb4c...

bors-servo added a commit that referenced this pull request Nov 25, 2019
Fix pre_scale on transform2d and transform3d

Fix the `pre_scale` method on both `Transform2D` and `Transform3D` plus added a test to make sure `pre_scale` and `post_scale` matched the equivalent sequence using full transforms.
@bors-servo
Copy link
Contributor

bors-servo commented Nov 25, 2019

☀️ Test successful - checks-travis
Approved by: nical
Pushing 759cb4c to master...

@bors-servo bors-servo merged commit b267280 into servo:master Nov 25, 2019
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
homu Test successful
Details
@nical nical mentioned this pull request Nov 25, 2019
bors-servo added a commit that referenced this pull request Nov 25, 2019
Version 0.20.5.

Contains the `pre_scale` fix from #377.
@jdm jdm mentioned this pull request Apr 7, 2020
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.