Increase Bezier timing function accuracy #428
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Test case? |
The polyfill evaluation of cubic Bezier timing functions gives slightly different results than the native Blink implementation. In my testing, this change reduced the absolute error by 63% while only increasing the number of cubic function evaluations by 31% This closes web-animations#139
ericwilligers
force-pushed
the
increase-bezier-accuracy
branch
from
1b2f076 to
83c9090
Compare
|
Test updated. |
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The polyfill evaluation of cubic Bezier timing functions
gives slightly different results than the native Blink
implementation.
In my testing, this change reduced the absolute error by 63%
while only increasing the number of cubic function evaluations
by 31%
This closes #139