-
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(dialog): Support default action button #3600
Conversation
f85f796
to
7d2e8c9
Compare
Codecov Report
@@ Coverage Diff @@
## master #3600 +/- ##
==========================================
+ Coverage 98.43% 98.43% +<.01%
==========================================
Files 120 120
Lines 5182 5192 +10
Branches 645 649 +4
==========================================
+ Hits 5101 5111 +10
Misses 81 81
Continue to review full report at Codecov.
|
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.
LGTM
All 545 screenshot tests passed for commit 621065d vs. |
All 557 screenshot tests passed for commit 6416429 vs. |
Fixes #3593.
Implements default action via a CSS modifier class that can be added to one of the buttons in the actions area of the dialog. If present, pressing enter while focused in the dialog will equate to clicking that button, unless:
click
does nothing)This can be tested live via the baseline-confirmation screenshot test page.
This adds an adapter API and changes another, but this is already encompassed under the general breaking change note for the new Dialog implementation.