WordPress.org

Make WordPress Core

Opened 11 months ago

Last modified 10 months ago

#39824 assigned defect (bug)

Gallery doesn't show images being uploaded

Reported by: gonom9 Owned by: adamsilverstein
Milestone: Future Release Priority: normal
Severity: normal Version: 4.0
Component: Gallery Keywords: has-patch reporter-feedback 2nd-opinion
Focuses: javascript Cc:

Description

When you insert a gallery with images being uploaded in Edit Post page, the gallery doesn't show the images.

How to Reproduce

  1. Open Edit Post page.
  2. Insert a gallery with one uploaded image.
  3. Select the gallery in tinyMCE then click on Edit.
  4. Upload a big image file and/or slow down the internet to buy time for the following actions.
  5. Choose Edit Gallery tab then click on Update Gallery button while the big image is uploading.
  6. The gallery shows an empty element instead of the image.
  7. If you edit the gallery and open Edit Gallery tab after the image uploaded, you'll see it has gone.

Attachments (1)

39824.diff (3.0 KB) - added by adamsilverstein 11 months ago.

Download all attachments as: .zip

Change History (4)

#1 @adamsilverstein
11 months ago

@gonom9 thanks for the bug report.

I verified this issue and captured a screencast to demonstrate the issue: https://cl.ly/3F0s1y0j021h

I have a potential solution I will upload shortly that disables the insert gallery button when you are in the process of uploading videos. I'm not certain this is the best approach, but it does prevent you from creating the broken condition you noted.

#2 @adamsilverstein
11 months ago

  • Focuses javascript added
  • Keywords has-patch reporter-feedback 2nd-opinion added
  • Version changed from 4.7.2 to 4.0

In 39824.diff:

Media: Disable the Insert/Update Gallery button while uploads are in progress.

Is there a better approach to resolving this issue?

@gonom9 can you give this a test and see if it resolves the issue for you?

#3 @adamsilverstein
10 months ago

  • Milestone changed from Awaiting Review to Future Release
  • Owner set to adamsilverstein
  • Status changed from new to assigned
Note: See TracTickets for help on using tickets.