-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 invertSelf to the DOMMatrix object #1649
Conversation
I have no idea what Mocha/Node wants me to do here to make the tests pass. I've used the exact same code and style as used in the rest of the tests, and yet it complains about a syntax error, where there literally are no syntax errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One other tiny suggestion while you're at it. Thanks for implementing this!
PR updated. |
@zbjornson? =) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Math checks out. Thanks 👏
This should address #1648 by adding the implementation for
invertSelf()
.inverse()
andinvertSelf()
added
section of the Changelog