ICEfaces
  1. ICEfaces
  2. ICE-1335

icefaces-extras.js not loaded on subsequent page renders

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6DR#2
    • Fix Version/s: 1.6DR#2, 1.6
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      JBoss 4.0.4, Firefox 1.5
    • Workaround Exists:
      Yes
    • Workaround Description:
      Put a UIComponent that uses effects in your UIComponent tree, outside of dynamic ui:includes, to force it to have the SCRIPT tag from the get-go.

      Description

      If you use the Facelets dynamic ui:include version of component-showcase, and click on the Effects node of the tree, then the effects won't work.

      Basically, the first page view of component-showcase does not include any UIComponents that require the icefaces-extras.js file, so it's rendered without that. Then, when the Effects page renders, it tells the framework to include that SCRIPT tag for icefaces-extras.js. But, when the DOM diff tries to work up the tree to find a parent node with an ID, so it can send that change to the browser, it can't since the HEAD tag has no ID. Plus, even if the HEAD tag did have an ID, updating it effectively removes the HEAD from the browser DOM, and messes everything up.

      So, it would be nice to find some way of only referencing the ~200KB icefaces-extras.js when effects are in use, but we might have to just force its usage for now, so that dynamic ui:include will work properly.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #13405 Mon Mar 19 16:59:01 MDT 2007 mircea.toma Always include ice-extra.js library -- ICE-1335 .
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/DOMResponseWriter.java
        Mark Collette created issue -
        Mark Collette made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Mark Collette made changes -
        Fix Version/s 1.6DR#2 [ 10040 ]
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #13414 Tue Mar 20 11:08:39 MDT 2007 mircea.toma Fix the library loading order -- ICE-1335.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/DOMResponseWriter.java
        Ken Fyten made changes -
        Fix Version/s 1.6 [ 10031 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Mircea Toma [ mircea.toma ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Mark Collette
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: