-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Popover dismiss fails on touchscreens #5099
Popover dismiss fails on touchscreens #5099
Comments
Do you find the same issue happens with iOS Safari (as Chrome on iOS uses Safari as it's rendering engine) |
It might be because iOS versions of the browsers do not "focus" buttons when they are clicked, and hence the blur (focusout) event doesn't get fired on the trigger element (button) https://bugs.webkit.org/show_bug.cgi?id=22261 |
@stevelacey could you see if PR #5103 fixes your issue? https://deploy-preview-5103--bootstrap-vue.netlify.com/docs/directives/popover#dismiss-on-next-click-self-dismissing |
@tmorehouse yep can confirm that fix works 🙌🏼 |
Describe the bug
The
v-b-popover.click.blur
solution for dismiss on next click (self dismissing) works fine on desktop but fails on touchscreen.Expected behavior
Tapping another popover trigger, or away from the popover should dismiss the popover.
Versions
Libraries:
Environment:
Screenshot:
This screenshot demonstrates the behaviour occurring on the demo in the documentation at https://bootstrap-vue.org/docs/directives/popover/#dismiss-on-next-click-self-dismissing
The text was updated successfully, but these errors were encountered: