Gallery doesn't show images being uploaded

[gonom9]

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.

[adamsilverstein]

@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.

[adamsilverstein]

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?

[adamsilverstein]

39824.2.diff​: refresh patch against trunk.

[adamsilverstein]

Testing the patch, it is not working as expected. during the upload process models are note getting the 'uploading' attribute set to true (instead it is undefined, while for uploaded posts it is false). This seems like a regression and will take some more digging to see why this is happening.

[adamsilverstein]

I still see a similar issue in WP 5.3 with the block editor: if you create a gallery, you can drop images onto the modal and then close it before uploads complete. Reopening media and editing the Gallery will show the upload progress, and update the gallery with completed items. Screencast reproducing issue in 5.3: ​https://share.getcloudapp.com/2NurEe2L

The original solution of disabling the "Insert/Update Gallery" button while uploads are in progress still feels right and would be a nice improvement. I'm going to revisit this for 5.4.

[audrasjb]

Hi there,

With Beta 3 approaching, we'll need a decision and a commit action very soon.
For Component maintainers: if you don't feel the current patch is ready to land in WP 5.4 in the next few day, it's could be better to move it to 5.5. We're leaving it in the milestone for now.

[lukecarbis]

As per @audrasjb's comment above, we're in beta 3 of 5.7 now. Moving to Future Release.