-
-
Notifications
You must be signed in to change notification settings - Fork 166
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 multi selecting blocks without click #4349
Conversation
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.
It works great!
f545233
to
5db1d6c
Compare
I rebased and moved the event registration to the created method to only register it once. Otherwise it would have been registered multiple times on every select call. @distantnative @afbora could you have another look at it? |
Thank you @bastianallgeier for improvement 💛 Still works great! I leave the technical review to @distantnative |
4b6a06b
5db1d6c
to
4b6a06b
Compare
This PR …
The issue fixed with listening
window.addEventListener('blur')
event and reset the multi select mode.Fixes
Breaking changes
none
Ready?
[ ] Unit tests for fixed bug/featureFor review team
[ ] Add to website docs release checklist (if needed)