-
Notifications
You must be signed in to change notification settings - Fork 65
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
Handling accidental double-taps on buttons with MMSingleClickMouseArea #3532
Conversation
Pull Request Test Coverage Report for Build 9991957790Details
💛 - Coveralls |
b8fce5a
to
cb42a0d
Compare
e9ea4f5
to
8004949
Compare
cb42a0d
to
f8f603b
Compare
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.
This is looking good now, thanks! I found one more place that would benefit from this new component. Can we please add it when opening child feature in relation form editors? It is again in both relation form editors when calling openLinkedFeature
signal.
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.
One last remaining place and I think we are good to go :)
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.
Thanks for all the updates!
New component
MMSingleClickMouseArea
for single-click detection. It temporarily disables further clicks using a Timer to prevent multiple triggers shortly after the first click.Fixes #3012