-
Notifications
You must be signed in to change notification settings - Fork 127
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
Modal Component #2795
base: main
Are you sure you want to change the base?
Modal Component #2795
Conversation
🦋 Changeset detectedLatest commit: 90fc87c The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Size Change: +6.83 kB (+1.02%) Total Size: 678 kB
ℹ️ View Unchanged
|
2923b4c
to
2b00b81
Compare
2b00b81
to
ddb9966
Compare
5b67511
to
7b1cc74
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2795 +/- ##
==========================================
+ Coverage 87.75% 87.83% +0.08%
==========================================
Files 221 225 +4
Lines 12746 13004 +258
Branches 1730 1776 +46
==========================================
+ Hits 11185 11422 +237
- Misses 1506 1528 +22
+ Partials 55 54 -1
|
7b1cc74
to
4a08192
Compare
4a08192
to
16d765e
Compare
cbc7308
to
dc6e5f0
Compare
dc6e5f0
to
6b2ccd8
Compare
bb7c9a2
to
adc0e75
Compare
aac8f8e
to
7943586
Compare
1271e93
to
0c7f118
Compare
0c7f118
to
bf25916
Compare
bf25916
to
a56459a
Compare
/preview |
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.
Big, big kudos for persevering through this complex refactor, your attention to detail and extensive testing 👏🏻👏🏻👏🏻
/preview |
/preview |
e982a3d
to
5266f90
Compare
5266f90
to
95af0e8
Compare
95af0e8
to
90fc87c
Compare
Addresses DSYS-860.
Purpose
Rewrite the modal component using the native HTML
<dialog/>
element with thedialog-polyfill
Approach and changes
react-modal
and use the<dialog/>
element in the modal component.useModal
hook.Definition of done