Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 1.8.2-EE-GA_P02
-
Fix Version/s: EE-1.8.2.GA_P03
-
Component/s: ICE-Components
-
Labels:None
-
Environment:n/a
-
Workaround Exists:Yes
-
Workaround Description:use the visible attribute instead of the rendered attribute
Description
When using a PanelPopup component, setting the rendered attribute from true to false causes all links on a page to become disabled. All links, even those outside of the rendered ICEfaces content, such as those in other portlets or a theme in a portal environment, become disabled. Only refreshing the page will re-enable the links. This seems to be due to some new Javascript code in style.js which 'deregisters' certain events on elements on the page when a modal overlay is activated. The events are 're-registered' when the modal overlay is deactivated, but this requires the PanelPopup to remain in the rendered state. If the popup is no longer rendered after the events are deregistered, they will not be reregisterd, even though the modal overlay is no longer active.
The following code in style.js Ice.modal.start() is responsible for the event deregistration, so this bug probably also applies to the other elements and events listed there, namely, 'input', 'select', 'textarea' and 'button'.
'input', 'select', 'textarea', 'button', 'a'].each(function(type) {
$enumerate(document.body.getElementsByTagName(type)).each(function(e) {
if (!childOfTarget(e)) {
var onkeypress = e.onkeypress;
var onkeyup = e.onkeyup;
var onkeydown = e.onkeydown;
var onclick = e.onclick;
e.onkeypress = none;
e.onkeyup = none;
e.onkeydown = none;
e.onclick = none;
rollbacks.push(function() {
try {
e.onkeypress = onkeypress;
e.onkeyup = onkeyup;
e.onkeydown = onkeydown;
e.onclick = onclick;
} catch (ex) {
logger.error('failed to restore callbacks on ' + e, ex);
}
});
}
});
The following code in style.js Ice.modal.start() is responsible for the event deregistration, so this bug probably also applies to the other elements and events listed there, namely, 'input', 'select', 'textarea' and 'button'.
'input', 'select', 'textarea', 'button', 'a'].each(function(type) {
$enumerate(document.body.getElementsByTagName(type)).each(function(e) {
if (!childOfTarget(e)) {
var onkeypress = e.onkeypress;
var onkeyup = e.onkeyup;
var onkeydown = e.onkeydown;
var onclick = e.onclick;
e.onkeypress = none;
e.onkeyup = none;
e.onkeydown = none;
e.onclick = none;
rollbacks.push(function() {
try {
e.onkeypress = onkeypress;
e.onkeyup = onkeyup;
e.onkeydown = onkeydown;
e.onclick = onclick;
} catch (ex) {
logger.error('failed to restore callbacks on ' + e, ex);
}
});
}
});
Issue Links
- is duplicated by
-
ICE-6323 Tabindex not changed when closing modal panelPopup
- Open
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #24270 | Mon Mar 28 16:34:45 MDT 2011 | mircea.toma | |
Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/bridge/lib/extras/style.js
|
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #24097 | Mon Mar 07 09:03:29 MST 2011 | tjohnson | Creating dir for 1.8.2 EE P02 |
Files Changed | ||||
ADD
/icefaces/scratchpads/patches/ICE-6333
|
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #24091 | Fri Mar 04 09:31:19 MST 2011 | mircea.toma | |
Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/bridge/src/application.js
MODIFY /icefaces/trunk/icefaces/bridge/lib/extras/style.js |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #23811 | Sun Jan 16 15:56:07 MST 2011 | mircea.toma | |
Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/bridge/lib/extras/style.js
|
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #23759 | Mon Jan 10 10:31:42 MST 2011 | mircea.toma | |
Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/panelpopup/PanelPopupRenderer.java
|