Modify

Ticket #322 (closed feature: fixed)

Opened 5 years ago

Last modified 23 months ago

Implement Video Upload Progress Bar

Reported by: anna Owned by:
Priority: critical Milestone: 4.5 EM and Plumi UI Re-Design
Component: VideoUploading Severity: New Ticket
Keywords: Cc:
Who will test this: And

Description (last modified by anna) (diff)

A flash/AJAX upload progress bar for uploading videos in Step 3 of the video publish form.

Attachments

error-file-upload-disappears.jpg Download (140.5 KB) - added by anna 2 years ago.

Change History

comment:1 Changed 4 years ago by and

  • Priority changed from major to critical
  • Who will test this set to And
  • Summary changed from Investigate Uploadify for progress bar to Investigate Uploadify or other option for progress bar

comment:2 Changed 4 years ago by and

  • Milestone changed from 3.1 to 4.1

Having to cull some stuff for 3.1. This is a big one so might have to wait. Again, anyone can feel free to grab these tickets - EM/Unweb can just not take them on.

comment:3 Changed 4 years ago by anna

  • Milestone changed from 4.3 to 4.1 Beta

comment:4 Changed 4 years ago by anna

  • Description modified (diff)

comment:5 Changed 4 years ago by markos

this one seems interesting:  http://plupload.com/

comment:6 Changed 4 years ago by markos

this is much simpler and elegant:  http://valums.com/ajax-upload/

comment:8 Changed 4 years ago by anna

  • Summary changed from Investigate Uploadify or other option for progress bar to Implement Video Upload Progress Bar
  • Description modified (diff)
  • Milestone changed from 4.1 Beta 1 (New Features and Fixes) to 4.2 Beta (Features and Refactoring)

As I haven't been able to test this yet, and we're getting close to our final deadline, I am bumping this to 4.2 milestone. If it turns out that its really close and you can still include it in the 4.1 release, feel free, but if it will create any delays at all we would prefer to meet deadlines than include this.

NOTES FROM PREVIOUS DESCRIPTION - UPLOADIFY

 http://plone.org/products/collective.uploadify

The test site for Critical Commons is not running so I can't show you the Uploadify progress bar in action, but we seemed to implement it without issues, after squashing some very nasty browser-crashing bugs.

The changes Andy made when implementing using collective.uploadify are available in the CC svn.

 http://svn.plone.org/svn/collective/criticalcommons-buildout/tags/beta3/src/collective.uploadify/

 http://svn.plone.org/svn/collective/criticalcommons-buildout/tags/beta6-1/products/CriticalCommons/skins/criticalcommons_templates/uploadify_plumi.js

comment:9 Changed 4 years ago by anna

  • Priority changed from critical to blocker

comment:10 Changed 3 years ago by and

  • Priority changed from blocker to major

comment:11 Changed 3 years ago by dimo

  • Milestone changed from 4.3 Beta1 to 4.4 Beta (Language Related New Features and Fixes)

comment:12 Changed 3 years ago by anna

  • Milestone changed from 4.6 Beta (Language Related New Features and Fixes) to 4.4 EM Re-Design

This is a huge priority for usability. Moving to 4.4 for discussion on best way forward.

comment:13 Changed 3 years ago by and

My only concern here (as much as I want this) is that this could take a lot of hours to develop as a feature, rather than redesigning with the features we have.

comment:14 Changed 3 years ago by anna

  • Type changed from task to feature

comment:15 Changed 3 years ago by anna

  • Milestone changed from 4.4 EM Re-Design to 4.3.2

comment:16 Changed 3 years ago by anna

  • Priority changed from major to critical

comment:17 Changed 3 years ago by cpsaltis

  • Milestone changed from 4.3.2 to 4.3.1

comment:18 Changed 3 years ago by anna

  • Milestone changed from 4.3.1 to 4.4 EM & Plumi UI Re-Design

We can now do this as part of building the new publish form.

comment:19 Changed 2 years ago by anna

This works very well :) :) :)

Except when there is an error on the page due to input from the user in the form.

When the user fails to input into a required field, or there is incorrect input (as per the URL field in the Categories tab), and the user clicks Save Changes, the file upload disappears from the progress bar.

The widget asks the user to again "Add a video file".

In actual fact, it seems like the file is added to the object successfully, but the user is unaware of this, thinking they have to add another video file.

The widget should continue to report that the file has been uploaded successfully.

comment:20 Changed 2 years ago by anna

See screenshot.

Changed 2 years ago by anna

comment:21 Changed 2 years ago by dimo

  • Milestone changed from 4.4 .Beta - Video Publishing to 4.4 .RC

comment:22 Changed 2 years ago by anna

Will test again today, apparently these issues are resolved and were to do with deployment.

comment:23 Changed 2 years ago by anna

Another issue regarding user interaction...

when the user clicks "Save Changes" before the video file has finished uploading, the upload process is cancelled.

so.. options for addressing this problem...

  1. we could have text "Please wait until the file has uploaded before clicking Save Changes"
  2. we could make the "Save Changes" button unavailable (greyed out, inactive) until the file has uploaded
  3. let this part of the object be saved at any time, asynchronously to the video file upload

thoughts?

comment:24 Changed 2 years ago by anna

Note: the page does report "Info: No file was uploaded"

so we could change this text..

comment:25 Changed 2 years ago by anna

the original problem listed above is indeed resolved :) the file remains associated with the object successfully.

comment:26 Changed 2 years ago by anna

actually, i just tested this again, the original problem remains. at least, it occurred again.

i clicked upload video file, the file uploaded. i entered a an invalid URL and clicked Save Changes. the first time, this worked fine, the page reported an error, but the video file was still there.

however, after some time had passed, i entered another URL, and clicked Save Changes. this time the video file was gone, with the message "Info: No file was uploaded."

comment:27 Changed 2 years ago by anna

re-testing this on testing.plumi.org

comment:28 Changed 2 years ago by anna

re-testing on new.plumi.org

comment:29 Changed 2 years ago by anna

okay, so some of these issues appear to now be resolved (after dimo had to make some changes due to the way the multiple zope instances were handling the upload sessions).

remaining issues:

  • 1. user can interfere with upload process by clicking save changes before file has uploaded so.

insert text "Please wait until the file has uploaded before clicking Save Changes" make the "Save Changes" button unavailable (greyed out, inactive) while a file is being uploaded

  • 2. at times the upload progress bar behaves oddly - the bar appears to flicker backwards and forwards during upload
  • 3. once after clicking Cancel on the progress bar, the file kept uploading, but the text in the bar said "Add a video file" (the upload did not cancel correctly

on the whole, when the user follows a simple linear progression, of clicking publish, adding a video file, waiting until it is uploaded, and then filling out fields and clicking Save Changes, these problems do not seem to occur.

comment:30 Changed 2 years ago by anna

  • Milestone changed from 4.4 .RC to 4.5 EM & Plumi UI Re-Design

Moving this to 4.5 to address these small bugs as part of the redesign.

comment:31 Changed 2 years ago by anna

Needs some more testing to iron out any little bugs for Plumi 4.5 release.

comment:32 Changed 23 months ago by anna

  • Status changed from new to closed
  • Resolution set to fixed
  • Severity set to New Ticket

Resolving this, any bugs to be reported individually.

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.