ICEfaces
  1. ICEfaces
  2. ICE-6204

ACE fileEntry multi-file upload

          Details

          • Type: New Feature New Feature
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Beta2
          • Fix Version/s: 4.0.BETA, 4.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            ACE
          • Assignee Priority:
            P2
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

            Description

            Instead of simply using an input type=file for file selection, display an editable table of selected files, each with their own progress bars, and file information. Each file selection may be deleted, and all may be deleted, before uploading. Maybe even find a way for the statuses to all show up for their respective file selection.

            When adding HTML 5 multiple file selection support, simply add the blocks of selections into the same table of selections.

              Activity

              Ascending order - Click to sort in descending order
              10 older comments
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Closing JIRA.

              Committed fixes to trunk for the four issues reported above at revision 38258: submitting the last selected file twice in FF; 'Add files' button not working after uploading files; 'Cancel upload' not correctly clearing the input type=file elements; and, re-uploading files every time 'Start upload' was clicked.

              The push issue is also fixed with another commit by Jack for PUSH-274.

              Show
              Arturo Zambrano added a comment - Closing JIRA. Committed fixes to trunk for the four issues reported above at revision 38258: submitting the last selected file twice in FF; 'Add files' button not working after uploading files; 'Cancel upload' not correctly clearing the input type=file elements; and, re-uploading files every time 'Start upload' was clicked. The push issue is also fixed with another commit by Jack for PUSH-274 .
              Hide
              Permalink
              Liana Munroe added a comment -

              Confirmed fixed Icefaces 3 trunk revision 38258, FF21, IE9, Chrome 29.

              Show
              Liana Munroe added a comment - Confirmed fixed Icefaces 3 trunk revision 38258, FF21, IE9, Chrome 29.
              Hide
              Permalink
              Liana Munroe added a comment -

              In IE 8 and 7 a script popup now occurs: see screen shot IE8FileEntry.PNG
              Line 5919
              Unknown runtime error.

              No issues seen in IE 9 or 10.

              Show
              Liana Munroe added a comment - In IE 8 and 7 a script popup now occurs: see screen shot IE8FileEntry.PNG Line 5919 Unknown runtime error. No issues seen in IE 9 or 10.
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Fixed javascript errors in IE7 and IE8 at revision 38295.

              The errors were caused by IE not supporting "innerHTML = '';". Changed approach to work with DOM methods to add and remove nodes. Also fixed other fixed table issues I found on IE7: not rendering the file list, not removing a cancelled file from list, and not removing the entire table.

              Some interesting facts about IE7:
              http://stackoverflow.com/questions/12613617/innerhtml-not-working-on-table-row-element-on-ie7
              http://stackoverflow.com/questions/812693/cant-dynamically-add-rows-to-a-table-in-ie
              http://stackoverflow.com/questions/17486889/internet-explorer-7-onclick-event-on-submit-button-does-not-fire
              http://stackoverflow.com/questions/95731/why-does-an-onclick-property-set-with-setattribute-fail-to-work-in-ie

              Testing notes: please re-test all browsers, since some general changes were made to fix these issues.

              Show
              Arturo Zambrano added a comment - Fixed javascript errors in IE7 and IE8 at revision 38295. The errors were caused by IE not supporting "innerHTML = '';". Changed approach to work with DOM methods to add and remove nodes. Also fixed other fixed table issues I found on IE7: not rendering the file list, not removing a cancelled file from list, and not removing the entire table. Some interesting facts about IE7: http://stackoverflow.com/questions/12613617/innerhtml-not-working-on-table-row-element-on-ie7 http://stackoverflow.com/questions/812693/cant-dynamically-add-rows-to-a-table-in-ie http://stackoverflow.com/questions/17486889/internet-explorer-7-onclick-event-on-submit-button-does-not-fire http://stackoverflow.com/questions/95731/why-does-an-onclick-property-set-with-setattribute-fail-to-work-in-ie Testing notes: please re-test all browsers, since some general changes were made to fix these issues.
              Hide
              Permalink
              Ken Fyten added a comment -

              Resolved.

              Show
              Ken Fyten added a comment - Resolved.

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Mark Collette
                • Votes:
                  3 Vote for this issue
                  Watchers:
                  7 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: