ICEfaces
  1. ICEfaces
  2. ICE-7258

Change default value for 'mandatoryResourceConfiguration' from null to 'none'

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.1-Beta, 2.1-Beta2, 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 2.1, Advanced Components Environment (ACE)
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration

      Description

      In ICEfaces 2.0.x the mandatoryResourceConfiguration parameter defaults to null, which results in all JavaScript and CSS resources for all components in the application being pre-loaded on each initial page load. This allows components to function correctly even if they are later added to the page dynamically during user interaction.

      Unfortunately, it also bloats each page load substantially (2+ MBs) and usually unnecessarily as most components are already on the page when it loads, and not added dynamically.

      To improve the out-of-the-box ICEfaces performance as it relates to page sizes and load times, we will change the default for mandatoryResourceConfiguration to 'none. This will result in no component resources being preloaded (only the resources for components on the page at initial load time will be loaded), resulting in an optimal resource load situation.

      This also means that any applications that are introducing ace components to the page dynamically (there are added to the page as part of a subsequent ajax page update and not during the initial page load) will now need to configure the mandatoryResourceConfiguration parameter (or preferably, the <icecore:config mandatoryResource=""> tag which can be specified per-view) in order to continue to function correctly.

        Issue Links

          Activity

          Repository Revision Date User Message
          ICEsoft Public SVN Repository #26014 Fri Oct 21 15:27:28 MDT 2011 ted.goddard merging in trunk changes for (ICE-7322) and (ICE-7258)
          Files Changed
          Commit graph MODIFY /icefaces2/tags/icefaces-core-2.1.0.BETA2/icefaces/core/src/main/java/org/icefaces/impl/event/BridgeSetup.java
          Commit graph MODIFY /icefaces2/tags/icefaces-core-2.1.0.BETA2/icefaces/core/src/main/javascript/application.js
          Commit graph MODIFY /icefaces2/tags/icefaces-core-2.1.0.BETA2/icefaces/core/src/main/java/org/icefaces/util/EnvUtils.java
          Commit graph MODIFY /icefaces2/tags/icefaces-core-2.1.0.BETA2/icefaces/core/src/main/java/org/icefaces/impl/renderkit/DOMRenderKit.java
          Commit graph MODIFY /icefaces2/tags/icefaces-core-2.1.0.BETA2/icefaces/core
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25937 Mon Oct 17 10:58:58 MDT 2011 ted.goddard removed dead code for mandatoryResource, implemented all checking in BridgeSetup (ICE-7258)
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/renderkit/DOMRenderKit.java
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/util/EnvUtils.java
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/BridgeSetup.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25648 Mon Sep 26 16:01:47 MDT 2011 ken.fyten ICE-7258 - Changed mandatoryResourceConfiguration default from null (load all) to 'none', load none.
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/util/EnvUtils.java

            People

            • Assignee:
              Ted Goddard
              Reporter:
              Ken Fyten
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: