Skip to content

fix: Image popover steals cursor focus #148

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

colombo15
Copy link

fixes #122

Moved the code to show the image popover to an onclick event on the img.

Added e2e test.

@netlify
Copy link

netlify bot commented Jun 29, 2022

Deploy Preview for nifty-lalande-39c157 ready!

Name Link
🔨 Latest commit 3a1591c
🔍 Latest deploy log https://app.netlify.com/sites/nifty-lalande-39c157/deploys/62bcf55de79eff000793d095
😎 Deploy Preview https://deploy-preview-148--nifty-lalande-39c157.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@b-kelly
Copy link
Collaborator

b-kelly commented Aug 30, 2022

Ah, somehow I lost track of this PR. Thank you for the submission! I'll review this as soon as possible.

@b-kelly b-kelly added bug Something isn't working mode - rich text Affects the editor's rich text (wysiwyg) mode labels Aug 30, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working mode - rich text Affects the editor's rich text (wysiwyg) mode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Image popover steals cursor focus
2 participants