ICEfaces
  1. ICEfaces
  2. ICE-8923

Minimize YUI usage in ACE

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 3.3
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            ICEfaces 3.x ACE components

            Description

            In order to minimize the amount of JS that the ACE components must load we would like to review the specific YUI functions used by the following ACE components and then try to establish a minimal subset of the overall YUI libraries to maintain/deploy in the ACE components.

            - ace:dataTable, paginator widget
            - ace:tabSet

            Modifying the underlying YUI JavaScript to enable a substantial reduction should also be considered.

              Activity

              Ascending order - Click to sort in descending order
              Ken Fyten created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Assignee Nils Lundquist [ nils.lundquist ]
              Fix Version/s 3.3 [ 10370 ]
              Assignee Priority P2 [ 10011 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33514 Tue Feb 19 13:21:21 MST 2013 nils.lundquist ICE-8923 - Removed formOf usage from accordion JS
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/accordion/accordion.js
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33515 Tue Feb 19 14:11:45 MST 2013 nils.lundquist ICE-8923 - Make ckeditor.mapping.js readable. Remove unused function from util.js.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/pushbutton/pushbutton.js
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/ckeditor.mapping.js
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/util/util.js
              Hide
              Permalink
              Nils Lundquist added a comment -

              With the removal of the YUI button components, the only component still using YUI via the ICEfaces API is ace:tabSet.

              ace:dataTable uses YUI Paginator directly.

              ckeditor / ace:richTextEntry doesn't use YUI.

              Show
              Nils Lundquist added a comment - With the removal of the YUI button components, the only component still using YUI via the ICEfaces API is ace:tabSet. ace:dataTable uses YUI Paginator directly. ckeditor / ace:richTextEntry doesn't use YUI.
              Nils Lundquist made changes -
              Summary Rationalize and consolidate YUI function usage to minimize footprint Remove YUI from ace:dataTable paginator
              Nils Lundquist made changes -
              Fix Version/s 3.3 [ 10370 ]
              Priority Major [ 3 ] Minor [ 4 ]
              Description In order to minimize the amount of JS that the ACE components must load we would like to review the specific YUI functions used by the following ACE components and then try to establish a minimal subset of the overall YUI libraries to maintain/deploy in the ACE components.

              - ace:dataTable, paginator widget
              - ace:tabSet

              Modifying the underlying YUI JavaScript to enable a substantial reduction should also be considered.

              		This JIRA captures the work required to create a new ace:dataTable paginator which will provide the same features and interface as the YUI component, without the dependencies and overhead.

              This is being done to isolate the remaining use of YUI in ACE to ace:tabSet.

              Original Description:
              In order to minimize the amount of JS that the ACE components must load we would like to review the specific YUI functions used by the following ACE components and then try to establish a minimal subset of the overall YUI libraries to maintain/deploy in the ACE components.

              - ace:dataTable, paginator widget
              - ace:tabSet

              Modifying the underlying YUI JavaScript to enable a substantial reduction should also be considered.

              Assignee Priority P2 [ 10011 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33534 Wed Feb 20 14:19:19 MST 2013 nils.lundquist ICE-8923 - Fixed typo in accordion.js form detection
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/accordion/accordion.js
              Hide
              Permalink
              Ken Fyten added a comment -

              With the commit for rvn#33514 - ICE-8923 - Removed formOf usage from accordion JS, the accordion no longer functions in showcase.

              INFO: 
ICEsoft Technologies Inc.
ICEpush 3.3.0.SNAPSHOT
Build number: 715
Revision: 33514

Feb 20, 2013 2:00:35 PM com.sun.faces.context.ExternalContextImpl getMimeType
WARNING: JSF1091: No mime type could be found for file http://wiki.icefaces.org/display/ICE/Accordion.  To resolve this, add a mime-type mapping to the applications web.xml.
Feb 20, 2013 2:00:35 PM com.sun.faces.context.ExternalContextImpl getMimeType
WARNING: JSF1091: No mime type could be found for file http://wiki.icefaces.org/display/ICE/AccordionPane.  To resolve this, add a mime-type mapping to the applications web.xml.
              Show
              Ken Fyten added a comment - With the commit for rvn#33514 - ICE-8923 - Removed formOf usage from accordion JS, the accordion no longer functions in showcase. INFO: ICEsoft Technologies Inc. ICEpush 3.3.0.SNAPSHOT Build number: 715 Revision: 33514 Feb 20, 2013 2:00:35 PM com.sun.faces.context.ExternalContextImpl getMimeType WARNING: JSF1091: No mime type could be found for file http: //wiki.icefaces.org/display/ICE/Accordion. To resolve this , add a mime-type mapping to the applications web.xml. Feb 20, 2013 2:00:35 PM com.sun.faces.context.ExternalContextImpl getMimeType WARNING: JSF1091: No mime type could be found for file http: //wiki.icefaces.org/display/ICE/AccordionPane. To resolve this , add a mime-type mapping to the applications web.xml.
              Hide
              Permalink
              Nils Lundquist added a comment -

              Fixed.

              Revision #33534
              Committed by nils.lundquist
              A minute ago
              ICE-8923 - Fixed typo in accordion.js form detection

              Show
              Nils Lundquist added a comment - Fixed. Revision #33534 Committed by nils.lundquist A minute ago ICE-8923 - Fixed typo in accordion.js form detection
              Nils Lundquist made changes -
              Description This JIRA captures the work required to create a new ace:dataTable paginator which will provide the same features and interface as the YUI component, without the dependencies and overhead.

              This is being done to isolate the remaining use of YUI in ACE to ace:tabSet.

              Original Description:
              In order to minimize the amount of JS that the ACE components must load we would like to review the specific YUI functions used by the following ACE components and then try to establish a minimal subset of the overall YUI libraries to maintain/deploy in the ACE components.

              - ace:dataTable, paginator widget
              - ace:tabSet

              Modifying the underlying YUI JavaScript to enable a substantial reduction should also be considered.

              		In order to minimize the amount of JS that the ACE components must load we would like to review the specific YUI functions used by the following ACE components and then try to establish a minimal subset of the overall YUI libraries to maintain/deploy in the ACE components.

              - ace:dataTable, paginator widget
              - ace:tabSet

              Modifying the underlying YUI JavaScript to enable a substantial reduction should also be considered.

              Nils Lundquist made changes -
              Summary Remove YUI from ace:dataTable paginator Minimize YUI usage in ACE
              Hide
              Permalink
              Nils Lundquist added a comment -

              Resolved- remaining effort abstracted to ICE-9008 for completion at some point beyond 3.3

              Show
              Nils Lundquist added a comment - Resolved- remaining effort abstracted to ICE-9008 for completion at some point beyond 3.3
              Nils Lundquist made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 3.3 [ 10370 ]
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Ken Fyten
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: