ICEfaces
  1. ICEfaces
  2. ICE-5886

Add framework support for dynamically coalescing multiple .js/.css file resources into a single .js/.css request/response

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Glimmer
    • Assignee Priority:
      P2
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      ISSUE: The current number of @ResourceDependency entries required for ACE-based components such as Slider is substantial, resulting in a large number of HTTP requests for a simple component, which can greatly increase page loading times.

        Issue Links

          Activity

          Hide
          Mircea Toma added a comment -

          Coalesced CSS resources are referenced first in the head (before the JS coalesced resources).

          Show
          Mircea Toma added a comment - Coalesced CSS resources are referenced first in the head (before the JS coalesced resources).
          Hide
          Ken Fyten added a comment -

          Re-opened. This is working well in general now.

          However, I noticed that with...

          <context-param>
              <param-name>org.icefaces.coalesceResources</param-name>
              <param-value>true</param-value>
          </context-param>
          

          ...about half the time the ace:chart demo will not function in showcase. JS errors abound in this case. Seems strange as sometimes it will work, and others not. I'll attach a screenshot of the errors.

          Show
          Ken Fyten added a comment - Re-opened. This is working well in general now. However, I noticed that with... <context-param> <param-name>org.icefaces.coalesceResources</param-name> <param-value> true </param-value> </context-param> ...about half the time the ace:chart demo will not function in showcase. JS errors abound in this case. Seems strange as sometimes it will work, and others not. I'll attach a screenshot of the errors.
          Hide
          Ken Fyten added a comment -

          JS errors when chart doesn't function.

          Show
          Ken Fyten added a comment - JS errors when chart doesn't function.
          Hide
          Mircea Toma added a comment -

          The fix makes sure that the coalesced resource is always referenced before the resources that are added later on.

          Show
          Mircea Toma added a comment - The fix makes sure that the coalesced resource is always referenced before the resources that are added later on.
          Hide
          Ken Fyten added a comment -

          Enabled resource coalescing by default in the showcase sample application.

          svn commit -m "ICE-5886 - Enable resource coalescing by default in the showcase sample application." 1 packet
          [16:34:34] - Sending /Users/Ken/Code/svn/icefaces3-trunk/icefaces/samples/showcase/showcase/src/main/webapp/WEB-INF/web.xml
          [16:34:35] - Committed revision 34369.

          Show
          Ken Fyten added a comment - Enabled resource coalescing by default in the showcase sample application. svn commit -m " ICE-5886 - Enable resource coalescing by default in the showcase sample application." 1 packet [16:34:34] - Sending /Users/Ken/Code/svn/icefaces3-trunk/icefaces/samples/showcase/showcase/src/main/webapp/WEB-INF/web.xml [16:34:35] - Committed revision 34369.

            People

            • Assignee:
              Mircea Toma
              Reporter:
              Ken Fyten
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: