-
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
feat(chips): Consolidate interaction event handlers #5251
Conversation
All 731 screenshot tests passed for commit 3f7af7f vs. |
Codecov Report
@@ Coverage Diff @@
## master #5251 +/- ##
=========================================
Coverage ? 98.63%
=========================================
Files ? 122
Lines ? 6002
Branches ? 772
=========================================
Hits ? 5920
Misses ? 81
Partials ? 1
Continue to review full report at Codecov.
|
All 731 screenshot tests passed for commit d7225b7 vs. |
Consolidate interaction event handlers into discrete root-level handlers:
handleClick
andhandleKeydown
.BREAKING CHANGE: the
handleInteraction
andhandleTrailingIconInteraction
handlers have been removed from theMDCChipFoundation
. ThehandleClick
handler has been added to theMDCChipFoundation