Fluid Infusion

[Uploader 2] When adding a file fails there is no user message

Details

  • Type: Bug Bug
  • Status: Reopened Reopened
  • Priority: Critical Critical
  • Resolution: Unresolved
  • Affects Version/s: 1.0, 1.1, 1.1.1, 1.1.2, 1.2
  • Fix Version/s: 1.4
  • Component/s: Uploader
  • Description:
    Hide
    In Uploader 2, browse for a file that is larger than the 20MB limit, and click OK

    Notice: that when the dialog dismisses that there is no message that the file was not added to your file queue.

    Fix will need to account for both files that fail because of file size and files that file because of type (there may be others, I will research)
    Show
    In Uploader 2, browse for a file that is larger than the 20MB limit, and click OK Notice: that when the dialog dismisses that there is no message that the file was not added to your file queue. Fix will need to account for both files that fail because of file size and files that file because of type (there may be others, I will research)
  • Environment:
    Win XP, IE7 (but probably all environments)

Issue Links

Activity

Hide
Eli Cochran added a comment - 19/May/09 3:21 AM
Added error messages for when a file queue fails.

Please note that there is some strange behavior in SWFUpload:
if the limit on the number of files that can be uploaded is 3 and the user attempts to select 4 files, instead of queing the first 3 files and then throwing an error, SWFUpload just throws an error and no files are queued. Fun!
Show
Eli Cochran added a comment - 19/May/09 3:21 AM Added error messages for when a file queue fails. Please note that there is some strange behavior in SWFUpload: if the limit on the number of files that can be uploaded is 3 and the user attempts to select 4 files, instead of queing the first 3 files and then throwing an error, SWFUpload just throws an error and no files are queued. Fun!
Hide
Eli Cochran added a comment - 19/May/09 9:44 AM
Forgot a few things in my check in:
- linting
- check in tests

Show
Eli Cochran added a comment - 19/May/09 9:44 AM Forgot a few things in my check in: - linting - check in tests
Hide
Justin Obara added a comment - 21/May/09 10:00 AM - edited
In case this isn't part of the "check in tests" from the comment above, the Uploader unit tests are currently broken.
http://build.fluidproject.org/infusion/tests/component-tests/uploader/html/Uploader-test.html
Show
Justin Obara added a comment - 21/May/09 10:00 AM - edited In case this isn't part of the "check in tests" from the comment above, the Uploader unit tests are currently broken. http://build.fluidproject.org/infusion/tests/component-tests/uploader/html/Uploader-test.html
Hide
Eli Cochran added a comment - 21/May/09 5:52 PM
I broke the tests when I failed to copy the HTML to the tests. Wondering if the tests should use AJAX to pull in the HTML the way that the Image Gallery does... wondering if that would even work.
Show
Eli Cochran added a comment - 21/May/09 5:52 PM I broke the tests when I failed to copy the HTML to the tests. Wondering if the tests should use AJAX to pull in the HTML the way that the Image Gallery does... wondering if that would even work.
Hide
Justin Obara added a comment - 22/May/09 9:11 AM
I can't comment on whether or not this is the correct approach. I believe that the UI Options test may do this though ( http://build.fluidproject.org/infusion/tests/component-tests/uiOptions/html/UIOptions-test.html ). The side effect is that the stats for "x tests of y failed" and the time to complete, don't work.
Show
Justin Obara added a comment - 22/May/09 9:11 AM I can't comment on whether or not this is the correct approach. I believe that the UI Options test may do this though ( http://build.fluidproject.org/infusion/tests/component-tests/uiOptions/html/UIOptions-test.html ). The side effect is that the stats for "x tests of y failed" and the time to complete, don't work.
Hide
Eli Cochran added a comment - 22/May/09 1:36 PM
Added basic unit tests and fixed some minor esthetic issues with IE
Show
Eli Cochran added a comment - 22/May/09 1:36 PM Added basic unit tests and fixed some minor esthetic issues with IE
Hide
Colin Clark added a comment - 25/May/09 3:28 PM
Eli missed a stray line of code here that causes a global variable to be defined with SWFObject. Easy fix.
Show
Colin Clark added a comment - 25/May/09 3:28 PM Eli missed a stray line of code here that causes a global variable to be defined with SWFObject. Easy fix.
Hide
Colin Clark added a comment - 25/May/09 6:04 PM
Bigger issues than I thought. On looking at this feature more closely, two show stoppers emerged:

1. The queue error dialog isn't implemented using jQuery UI's Dialog widget, so it lack a number of the keyboard accessibility fixes we added. As a result, focus is not captured within the dialog, and in fact moves directly behind the dialog.

2. There are no ARIA roles or states that would inform an AT user that this is an actual dialog.

We're not going to have a chance to implement fixes to these accessibility blockers before code freeze, so I've moved the code into a branch and removed the feature from trunk in preparation for the 1.1 release.
Show
Colin Clark added a comment - 25/May/09 6:04 PM Bigger issues than I thought. On looking at this feature more closely, two show stoppers emerged: 1. The queue error dialog isn't implemented using jQuery UI's Dialog widget, so it lack a number of the keyboard accessibility fixes we added. As a result, focus is not captured within the dialog, and in fact moves directly behind the dialog. 2. There are no ARIA roles or states that would inform an AT user that this is an actual dialog. We're not going to have a chance to implement fixes to these accessibility blockers before code freeze, so I've moved the code into a branch and removed the feature from trunk in preparation for the 1.1 release.
Hide
Anastasia Cheetham added a comment - 29/May/09 3:22 PM
In testing for the 1.1 release, I still see this issue - there is no indication that an error occurred, other than the fact that the file doesn't show up in the queue.
Show
Anastasia Cheetham added a comment - 29/May/09 3:22 PM In testing for the 1.1 release, I still see this issue - there is no indication that an error occurred, other than the fact that the file doesn't show up in the queue.

People

  • Assignee:
    Unassigned
    Reporter:
    Eli Cochran
  • Votes:
    0
    Watchers:
    0

Dates

  • Created:
    17/Nov/08 12:46 PM
    Updated:
    15/Apr/10 11:41 AM