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 event-listeners during dismiss click #36863
Conversation
bdb569b
to
d6c2aab
Compare
9300215
to
a41d71f
Compare
a41d71f
to
d710920
Compare
Is this good to merge @GeoSot or do you need a review from someone? |
I would appreciate a review just to be sure |
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.
I prefer this one compared to the alternative approach.
Tested the whole Modals page on macOS with different browsers (Firefox, Chrome, Edge and Safari) and couldn't spot any regressions.
LGTM! 🚀
d710920
to
cf6b611
Compare
cf6b611
to
3d0c07b
Compare
@GeoSot Any 5.2.2 release ? Can you disable 5.2.1 because it broke many projets. Have to force 5.2.0 for now. |
Yes, please do a 5.2.2 soon. Thank you! |
I can understand you on this. I personally, have done a lot of refactoring to bring our codebase to a better condition, preparing it for v6, and unfortunately sometimes mistakes can be done. I apologize for this. |
I will release 5.2.2 tomorrow or Monday. Please be patient.
…On Sat, Oct 1, 2022, 19:58 GeoSot ***@***.***> wrote:
I can understand you on this. I personally, have done a lot of refactoring
to bring our codebase to a better condition, preparing it for v6, and
unfortunately sometimes mistakes can be done. I apologize for this.
Furthermore, as a team, we are trying our best, working on our free time,
so please make some patience and we will try to have a release soon
—
Reply to this email directly, view it on GitHub
<#36863 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACVLNOTTBSVQK2ZRGD3XRDWBBUUNANCNFSM55CIOTHA>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
Partially regression of #36401.
Fixes modal listeners in order to:
mousedown
,mouseup
listen to scrollbar clickclick
ignores scrollbar click.modal-dialog
, but ends outside of it.modal-dialog
closes: #36855
alternative approach #36855 (comment)