Skip to content
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

Can't replace image in gallery block #30158

Closed
tomdkat opened this issue Mar 23, 2021 · 8 comments
Closed

Can't replace image in gallery block #30158

tomdkat opened this issue Mar 23, 2021 · 8 comments
Labels
[Block] Gallery Affects the Gallery Block - used to display groups of images [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed [Type] Bug An existing feature does not function as intended

Comments

@tomdkat
Copy link

tomdkat commented Mar 23, 2021

Description

After adding images to a 3-column gallery block, I'm unable to replace an image with another image. I can click the image I want to replace and I can click the "Pencil" icon to make changes, but the "upload", "media library", and "Insert from URL" links are not clickable.

Step-by-step reproduction instructions

  1. Create a page and add a 3-column gallery
  2. Add three images to the 3-column gallery, with one image per column
  3. Save the page
  4. Edit the 3-column gallery by choosing the middle photo
  5. Click the "Pencil" icon to make changes to the photo
  6. When the "Upload", "Media Library", and "Insert from URL" links appear, try to click one

Expected behaviour

The "Upload", "Media Library", and "Insert from URL" links should be clickable so a replacement photo can be selected.

Actual behaviour

The "Upload", "Media Library", and "Insert from URL" links are not clickable, so a replacement photo can't be selected.

Screenshots or screen recording (optional)

gutenberg-gallery-block-sample

Code snippet (optional)

N/A

WordPress information

  • WordPress version: 5.7.0
  • Gutenberg version: (whatever comes with Wordpress 5.7.0)
  • Are all plugins except Gutenberg deactivated? "No"
  • Are you using a default theme (e.g. Twenty Twenty-One)? "No" (I conducted a test using the Twenty Twenty-One theme and the issue was repeatable)

Device information

  • Device: Laptop
  • Operating system: "Windows 10" or Linux Mint 18.3
  • Browser: Firefox 86 or Chrome 89
@annezazu
Copy link
Contributor

I can replicate this using Gutenberg 10.2.1 and WordPress 5.7. However, I noticed the behavior is a bit inconsistent and "sticky" with the ability to use the buttons returning if you click on other images. Here's a video to better explain where you can see me clicking around trying to see what might be causing this :D

replicating.mov

@noisysocks this seems important to get fixed for 5.7.1 (April 14th release) but would love your thoughts as I don't trust my read of priority issues for releases.

@annezazu annezazu added [Block] Gallery Affects the Gallery Block - used to display groups of images [Type] Bug An existing feature does not function as intended labels Mar 23, 2021
@tomdkat
Copy link
Author

tomdkat commented Mar 24, 2021 via email

@annezazu
Copy link
Contributor

Whew, glad you experienced it too! Thanks for reporting it :) Big help to catch these problems early.

@Quintis1212
Copy link
Contributor

Quintis1212 commented Mar 27, 2021

Hello @annezazu and @tomdkat . I finded the source of bug . There is a before element that has absolute position which covers buttons , when position absolute is disabled all is working :
before-el

This before element is used for outline on hover . I think it will be better implement outline on hover with class such as with this component below:
correct

If I am right please tell me and I will implement this fix in PR )

@tomdkat
Copy link
Author

tomdkat commented Mar 27, 2021

Will the "Upload", "Media Library" and "Insert from URL" links be retained? Outside of that, your proposed solution sounds good to me. :) Thanks!

@Quintis1212
Copy link
Contributor

Will the "Upload", "Media Library" and "Insert from URL" links be retained? Outside of that, your proposed solution sounds good to me. :) Thanks!

Yes )

@tomdkat
Copy link
Author

tomdkat commented Mar 28, 2021 via email

@stokesman
Copy link
Contributor

It appears this is a duplicate of #29835 which has already been closed with the fix in #29860. The fix is scheduled to be included in the next minor release.

I'll close this one. It'd be nice if there was a way to help prevent this type of thing from happening. Though probably that's an issue for Github itself. I think the best way would be by inserting and refining a list of possible duplicates on the issue creation page as the title (and maybe description) are updated.

@stokesman stokesman added the [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed label Mar 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Gallery Affects the Gallery Block - used to display groups of images [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

4 participants