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

File upload indicator doesn't work for large files #369

Closed
westonian opened this issue Nov 20, 2014 · 11 comments
Closed

File upload indicator doesn't work for large files #369

westonian opened this issue Nov 20, 2014 · 11 comments

Comments

@westonian
Copy link

Upload a very large file (100MB) and watch the file progress indicator...

expected behavior - it does something...like shows upload progress...
current behavior - it shows 0% and then the UI flashes after sitting there for many seconds the file appears

@gaitonde
Copy link
Contributor

@kbowerma - fyi

@bryceglass
Copy link

@gaitonde @kbowerma @westonian

Turns out this is a problem for smaller / medium-size uploads as well -- progress bar goes basically from 0-100 w/no usable feedback. We've observed this with 3/3 of our informal user sessions (bruzzi, Nathan and Wes all experienced it - uncertainty about whether files were uploading or frozen.)

@gaitonde
Copy link
Contributor

@bryceglass - the upload progress bar for challenge submission works well. It's a different design/widget, but can we just use that instead?

cc/ @kbowerma @indytechcook @westonian

@bryceglass
Copy link

@dayal, yeah I think this'll be fine (as long as it displays legibly in the smaller footprint of the existing DIV..

image

cc/ @kbowerma @indytechcook @westonian

@gaitonde gaitonde added this to the 2014.12.04 milestone Nov 26, 2014
@gaitonde gaitonde removed the Proposed label Nov 26, 2014
@gaitonde
Copy link
Contributor

@bryceglass - can you look for an alternative component that is a spinner?

@gaitonde
Copy link
Contributor

@bryceglass - i would choose from one of these -

http://getbootstrap.com/components/#progress

@bryceglass
Copy link

I thought the short-term fix was to replace a 'progress' bar (showing degree of completion) w/a simple 'activity' (sh-t's happening) indicator?

How about we just put this in the DIV until the file is available?

<i class="fa fa-circle-o-notch fa-spin"></i>

See it in action here: http://fortawesome.github.io/Font-Awesome/examples/#spinning

@kbowerma
Copy link
Contributor

kbowerma commented Dec 3, 2014

@bryceglass this was an accident but I kind of like it. I am still working on getting the progress meter to actually work.

http://screencast.com/t/z5q9i9MFDj

@bryceglass
Copy link

@kbowerma HA! I kinda like it too! A happy little accident. I'd say we leave it, but be prepared for people to report it as a bug. ;-)

@kbowerma
Copy link
Contributor

kbowerma commented Dec 3, 2014

I think I got it fixed back to the original progress meter but I am throwing an error I need to figure out:
http://screencast.com/t/CMm4HKmMAy

kbowerma pushed a commit to appirio-tech/lc1-challenge-app that referenced this issue Dec 3, 2014
kbowerma pushed a commit to appirio-tech/lc1-challenge-app that referenced this issue Dec 3, 2014
@bryceglass
Copy link

That looks great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants