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(modal): prevent modal from closing when using scrollbar #3531
fix(modal): prevent modal from closing when using scrollbar #3531
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3531 +/- ##
==========================================
+ Coverage 91.39% 91.41% +0.02%
==========================================
Files 96 96
Lines 2800 2807 +7
Branches 517 518 +1
==========================================
+ Hits 2559 2566 +7
Misses 183 183
Partials 58 58
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
@maxokorokov thanks for addressing the problem!
b0af2e5
to
6635ecb
Compare
6635ecb
to
8bdb515
Compare
@maxokorokov when CI is ok, let's merge it 🚀 |
Using `mousedown` - `mouseup` - `click` combination to handle closing correctly. Fixes #3518
Have to use
mousedown
-mouseup
-click
to handle closing correctly:click
fixes the Modals dismissed by mouse release when trying to scroll the window #3518mousedown
-mouseup
fixes closing on accidental drags from modal dialog → backdropFixes #3518