Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 1.6DR#2
-
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.
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 -- |
Files Changed | ||||
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 -- |
Files Changed | ||||
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 ] |