-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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(text-field): Send client position to line ripple for touch events #4084
Conversation
All 655 screenshot tests passed for commit 0757e68 vs. |
target: { | ||
getBoundingClientRect: () => { | ||
return {}; | ||
const mockTouchStartEvent = { |
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.
Should we update the verify
call to expect a specific value passed to setLineRippleTransformOrigin
in order to better test this fix?
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.
Done.
Codecov Report
@@ Coverage Diff @@
## master #4084 +/- ##
==========================================
+ Coverage 98.67% 98.67% +<.01%
==========================================
Files 126 126
Lines 5601 5603 +2
Branches 746 747 +1
==========================================
+ Hits 5527 5529 +2
Misses 74 74
Continue to review full report at Codecov.
|
All 663 screenshot tests passed for commit 48ecc5e vs. |
Fixes #3643