ICEfaces
  1. ICEfaces
  2. ICE-6597

Improving Icefaces static resources (JavaScript and CSS)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 3.2
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Recent Ubuntu version with Glassfish v3; not necessary

      Description

      When using the recent version of the Icefaces ACE components (v2) a lot of static resources (hereinafter ICE resources) are embedded. In total 66 JavaScript and CSS files are embedded which make the system practically unusable. In total ~1mb has to be downloaded on the first request in order to display the page. After that caching is not optimal as ICE resources are only allowed to be cached for ten minutes.

        Issue Links

          Activity

          Hide
          Ben R added a comment - - edited

          Forgot to mention that are also some absolutely useless static resources included.

          An empty file: /javax.faces.resource/yui3.js.iface?ln=org.icefaces.component.util
          http://www.patternrepository.com/web-interface/javax.faces.resource/yui3.js.iface?ln=org.icefaces.component.util
          ---------------------------------------------
          /*

          • Version: MPL 1.1/GPL 2.0/LGPL 2.1
            *
          • "The contents of this file are subject to the Mozilla Public License
          • Version 1.1 (the "License"); you may not use this file except in
          • compliance with the License. You may obtain a copy of the License at
          • http://www.mozilla.org/MPL/
            *
          • Software distributed under the License is distributed on an "AS IS"
          • basis, WITHOUT WARRANTY OF ANY KIND, either express or implied. See the
          • License for the specific language governing rights and limitations under
          • the License.
            *
          • The Original Code is ICEfaces 1.5 open source software code, released
          • November 5, 2006. The Initial Developer of the Original Code is ICEsoft
          • Technologies Canada, Corp. Portions created by ICEsoft are Copyright (C)
          • 2004-2006 ICEsoft Technologies Canada, Corp. All Rights Reserved.
            *
          • Contributor(s): _____________________.
            *
          • Alternatively, the contents of this file may be used under the terms of
          • the GNU Lesser General Public License Version 2.1 or later (the "LGPL"
          • License), in which case the provisions of the LGPL License are
          • applicable instead of those above. If you wish to allow use of your
          • version of this file only under the terms of the LGPL License and not to
          • allow others to use your version of this file under the MPL, indicate
          • your decision by deleting the provisions above and replace them with
          • the notice and other provisions required by the LGPL License. If you do
          • not delete the provisions above, a recipient may use your version of
          • this file under either the MPL or the LGPL License."
            *
            */
            ---------------------------------------------

          A stylesheet which just contains an import command: /javax.faces.resource/slider.css.iface?ln=org.icefaces.component.sliderentry
          http://www.patternrepository.com/web-interface/javax.faces.resource/slider.css.iface?ln=org.icefaces.component.sliderentry
          ---------------------------------------------
          /* load core styling from YUI library */
          @import url('/web-interface/javax.faces.resource/yui/3_1_1/slider/assets/slider-core.css.iface');
          ---------------------------------------------

          Show
          Ben R added a comment - - edited Forgot to mention that are also some absolutely useless static resources included. An empty file: /javax.faces.resource/yui3.js.iface?ln=org.icefaces.component.util http://www.patternrepository.com/web-interface/javax.faces.resource/yui3.js.iface?ln=org.icefaces.component.util --------------------------------------------- /* Version: MPL 1.1/GPL 2.0/LGPL 2.1 * "The contents of this file are subject to the Mozilla Public License Version 1.1 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.mozilla.org/MPL/ * Software distributed under the License is distributed on an "AS IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License for the specific language governing rights and limitations under the License. * The Original Code is ICEfaces 1.5 open source software code, released November 5, 2006. The Initial Developer of the Original Code is ICEsoft Technologies Canada, Corp. Portions created by ICEsoft are Copyright (C) 2004-2006 ICEsoft Technologies Canada, Corp. All Rights Reserved. * Contributor(s): _____________________. * Alternatively, the contents of this file may be used under the terms of the GNU Lesser General Public License Version 2.1 or later (the "LGPL" License), in which case the provisions of the LGPL License are applicable instead of those above. If you wish to allow use of your version of this file only under the terms of the LGPL License and not to allow others to use your version of this file under the MPL, indicate your decision by deleting the provisions above and replace them with the notice and other provisions required by the LGPL License. If you do not delete the provisions above, a recipient may use your version of this file under either the MPL or the LGPL License." * */ --------------------------------------------- A stylesheet which just contains an import command: /javax.faces.resource/slider.css.iface?ln=org.icefaces.component.sliderentry http://www.patternrepository.com/web-interface/javax.faces.resource/slider.css.iface?ln=org.icefaces.component.sliderentry --------------------------------------------- /* load core styling from YUI library */ @import url('/web-interface/javax.faces.resource/yui/3_1_1/slider/assets/slider-core.css.iface'); ---------------------------------------------
          Hide
          Ken Fyten added a comment -

          We are aware of most of these issues and are working towards major improvements, current thinking is for 2.1.

          Show
          Ken Fyten added a comment - We are aware of most of these issues and are working towards major improvements, current thinking is for 2.1.
          Hide
          Mark Collette added a comment -

          This is basically a master task jira for optimisation. We should link in the new jiras for the coalescing of js and css for the new ACE components (we already did this fir the first batch of ACE components).

          Show
          Mark Collette added a comment - This is basically a master task jira for optimisation. We should link in the new jiras for the coalescing of js and css for the new ACE components (we already did this fir the first batch of ACE components).
          Hide
          Ken Fyten added a comment - - edited

          An other issue is that in the released version none of the static resources are optimized nor compressed which would save a significant amount of bandwidth.

          To summarize it:

          • use less static resources
          • optimize and compress them
          • use sprites
          • combine JavaScript and CSS files to two big files as this would save a lot of HTTP requests

          Is there any chance that this could be incorporated into a small release?

          was:
          When using the resent version of the Icefaces ACE components (v2) a lot of static resources (hereinafter ICE resources) are embedded. In total 66 JavaScript and CSS files are embedded which make the system practically unusable. In total ~1mb has to be downloaded on the first request in order to display the page. After that caching is not optimal as ICE resources are only allowed to be cached for ten minutes.

          An other issue is that in the released version none of the static resources are optimized nor compressed which would save a significant amount of bandwidth.

          To summarize it:

          • use less static resources
          • optimize and compress them
          • use sprites
          • combine JavaScript and CSS files to two big files as this would save a lot of HTTP requests

          Is there any chance that this could be incorporated into a small release?

          Show
          Ken Fyten added a comment - - edited An other issue is that in the released version none of the static resources are optimized nor compressed which would save a significant amount of bandwidth. To summarize it: use less static resources optimize and compress them use sprites combine JavaScript and CSS files to two big files as this would save a lot of HTTP requests Is there any chance that this could be incorporated into a small release? was: When using the resent version of the Icefaces ACE components (v2) a lot of static resources (hereinafter ICE resources) are embedded. In total 66 JavaScript and CSS files are embedded which make the system practically unusable. In total ~1mb has to be downloaded on the first request in order to display the page. After that caching is not optimal as ICE resources are only allowed to be cached for ten minutes. An other issue is that in the released version none of the static resources are optimized nor compressed which would save a significant amount of bandwidth. To summarize it: use less static resources optimize and compress them use sprites combine JavaScript and CSS files to two big files as this would save a lot of HTTP requests Is there any chance that this could be incorporated into a small release?
          Hide
          Ken Fyten added a comment - - edited

          Marking fixed as of 3.1.

          Show
          Ken Fyten added a comment - - edited Marking fixed as of 3.1.

            People

            • Assignee:
              Unassigned
              Reporter:
              Ben R
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: