Media Upload - Cancel Upload Button

[landwire]

What should this button actually do? It cancels the upload while a file is uploading (progress bar moving), but once you have chosen a few files and they all have uploaded, I change my mind and do not want to go ahead with the upload. I press "Cancel Upload", but nothing happens. It should cancel my upload though....

IMO, "Cancel Upload" button does exactly what it says: cancels the upload process. Once the images are uploaded, they can be explicitly deleted from Media Library if not needed.

Agree with Sergey. Cancel Upload should cancels uploads in progress. Deleting files because you changed your mind is not the same. That said, the Cancel Upload button seems to stay in active state even after crunching is finished, which does make it confusing, and it's inside the new uploader area. @azaozz: why is Cancel Upload still clickable when nothing is actively uploading?

I've also noticed that while "Cancel Upload" button immediately removes the progress bar (creating an impression that the upload is also immediately stopped), the file currently being uploaded silently finishes uploading anyway.

Replying to jane:

@azaozz: why is Cancel Upload still clickable when nothing is actively uploading?

Seems to be a bug from the initial integration of Plupload (GSoC) that went unnoticed until now... Fixing.

In [19099]:

Disable the Cancel Upload button after the upload has finished or if it was interrupted, fixes #19050

Replying to SergeyBiryukov:

I've also noticed that while "Cancel Upload" button immediately removes the progress bar (creating an impression that the upload is also immediately stopped), the file currently being uploaded silently finishes uploading anyway.

Replying to nacin:

Don't think you can stop the browser finishing the uploading of the current file from JS, but patches welcome.

See

• ​http://www.plupload.com/punbb/viewtopic.php?id=778
• ​http://www.plupload.com/punbb/viewtopic.php?id=207
• ​https://github.com/moxiecode/plupload/issues/299
• ​http://www.plupload.com/punbb/viewtopic.php?id=261 Maybe possible with chunked upload?

Replying to azaozz:

Don't think you can stop the browser finishing the uploading of the current file from JS

Maybe we can reflect that in UI (e.g. by not removing current progress bar)?

Replying to ocean90:

Maybe possible with chunked upload?

By default we are not using chunking, it doesn't work well in all browsers when using the HTML5 runtime that supports drag & drop.

On the other hand we could try to hook at the HXR and use abort(), not sure if that's exposed though.

Replying to SergeyBiryukov:

Maybe we can reflect that in UI (e.g. by not removing current progress bar)?

I'm starting to think we should drop the Cancel button completely. Other similar setups don't have "cancel" either (like G+). Users can cancel by closing the popup/reloading the page.

Replying to azaozz:

I'm starting to think we should drop the Cancel button completely. Other similar setups don't have "cancel" either (like G+). Users can cancel by closing the popup/reloading the page.

Fine by me.

It occurs to me that a small 'x' appearing in the loading bar only during the upload might be really helpful if for example someone mis-clicks and gets the wrong file.

Replying to olleicua:

The problem is that we can't cancel the currently uploading file from JS (for now). Only way would be to reload the page or close the popup.

In [19127]:

Remove the Cancel Upload button, fixes #19050