Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 1.6DR#4
-
Component/s: ICE-Components
-
Labels:None
-
Environment:WinXP / Tomcat 5.5.17 / Liferay 4.3.0 (pre-release)
Description
There are several problems:
1. The modal popup does not appear as the topmost window
2. The modal popup does not grey-out the rendered browser page under it
3. The modal popup window does not take control over the entire browser window
4. The draggable panel renders but is not draggable
5. The "Close" button on the draggable panel does not close the draggable panel
-
Hide
- icefaces-test-popup-portlet.zip
- 3.79 MB
- Neil Griffin
-
- icefaces-test-popup-portlet/.../context.xml 0.1 kB
- icefaces-test-popup-portlet/.../MANIFEST.MF 0.1 kB
- icefaces-test-popup-portlet/PopupTest.jspx 6 kB
- icefaces-test-popup-portlet/.../PopupTest.class 1 kB
- icefaces-test-popup-portlet/.../faces-config.xml 2 kB
- icefaces-test-popup-portlet/.../geronimo-web.xml 1 kB
- icefaces-test-popup-portlet/.../backport-util-concurrent.jar 343 kB
- icefaces-test-popup-portlet/.../commons-beanutils.jar 184 kB
- icefaces-test-popup-portlet/.../commons-collections.jar 546 kB
- icefaces-test-popup-portlet/.../commons-digester.jar 107 kB
- icefaces-test-popup-portlet/.../commons-fileupload.jar 52 kB
- icefaces-test-popup-portlet/.../commons-validator.jar 136 kB
- icefaces-test-popup-portlet/.../el-api.jar 24 kB
- icefaces-test-popup-portlet/.../icefaces-comps.jar 627 kB
- icefaces-test-popup-portlet/.../jsf-api.jar 356 kB
- icefaces-test-popup-portlet/.../jsf-impl.jar 778 kB
- icefaces-test-popup-portlet/.../just-ice.jar 641 kB
- icefaces-test-popup-portlet/.../oro.jar 64 kB
- icefaces-test-popup-portlet/.../util-bridges.jar 9 kB
- icefaces-test-popup-portlet/.../util-java.jar 278 kB
- icefaces-test-popup-portlet/.../util-taglib.jar 110 kB
- icefaces-test-popup-portlet/.../versions-licenses.html 4 kB
- icefaces-test-popup-portlet/.../apache-license-1.1-discovery.txt 2 kB
- icefaces-test-popup-portlet/.../apache-license-1.1-oro.txt 2 kB
- icefaces-test-popup-portlet/.../apache-license-2.0.txt 11 kB
- icefaces-test-popup-portlet/.../creative-commons-public-domain.html 2 kB
- icefaces-test-popup-portlet/.../jsf-1.1-api-license.txt 7 kB
- icefaces-test-popup-portlet/.../jsf-1.1-impl-license.txt 10 kB
- icefaces-test-popup-portlet/.../jsf-1.1-third-party-licenses.txt 31 kB
- icefaces-test-popup-portlet/.../jstl-1.1-license.txt 7 kB
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Forgot to mention that IE reports a JavaScript error on page load, but FireFox does not.
Here is the IE error message:
Line 295
Char 26
Error: 'Ice.modal' is null or not an object
Code: 0
The bug has been fixed -revision 13930
We have tested it under Servlets environment, it is working fine. However it needs to be tested in portlet environment.
Adnan,
It is still broken for portlets.
Deryk wrote via IM: "Some of our 'ice-extra.js' is not being applied"
with regard to portlets.
Neil
Seems like it is some different issue, the fix we had was for the following JIRA case:
http://jira.icefaces.org//browse/ICE-1598
Neil,
My apology, the fix was for some other issue, related to the panelPopup component. Dyerk is working on this issue.
Thanks,
Adnan
It appears that the ice-extras.js file is not being included. The logic for JavaScriptContext to add extra libraries (putting them in the HEAD) is not executed for portlets as they are treated as included fragments. Adding this library in seems to solve most of the issues outlined above. A possible work around is to manually add a script tag to the portlet page that needs this ice-extras.js code.
Okay...manually adding in the script tag doesn't work but some experimental code seemed to indicate that most of the problems are handled if the extras library is included so we'll concentrate on that.
The ice-extras.js file is now written out for all portlets all the time. It does not completely solve this problem but behaviour is better.
1. The modal popup does not appear as the topmost window - FIXED
2. The modal popup does not grey-out the rendered browser page under it - FIXED
3. The modal popup window does not take control over the entire browser window - FIXED
4. The draggable panel renders but is not draggable - FIXED
5. The "Close" button on the draggable panel does not close the draggable panel - NOT FIXED
In addition, having multiple instances of the popup panel seems to adversely effect the behaviour so perhaps multiple copies of the ice-extras.js file do not play well together. Further investigation is required.
Revising the subject line to be more generic.
So although it's not completely fixed, a couple of new issues have cropped up.
1) The modal dialog is completely disabled in IE.
2) Adding another portlet to the page causes popups to stop working properly.
FYI, a Liferay customer just wrote the following regarding a sample portlet I provided him that attempts to demonstrate the modal popup feature:
Thanks for the demo!
The dialog indeed is displayed but can't be closed and is not modal for me... So evidently it needs some work, hopefully ICEFaces team will figure it out... But it is a great start...
--Neil
I simplified the test case further and started with the non-modal popup. It seems that the culprit for the Close button not working on the draggable is the type="button" attribute. Removing this seems to make the "Close" button work again. In checking the 1.5.3 release, that attribute is in the component-showcase example for popups but has since been removed. The spec for command button says that the only valid values are "submit" (the default) and "reset". In any event, the draggable popup seems to work as advertised in the test portlet when the suggested changes is made.
The modal dialog code didn't have the type attributes set for the close button so didn't show the issue. I tried putting two instances of the test portlet on the same page and, once the above changes was made, both types of popup panels appeared to work as per the component showcase example. There may still be some work around how popups should be "owned" by the portlet that spawns them (e.g. should a modal popup be modal for the whole page or should it just be model for the parent portlet) but that discussion should probably be done elsewhere. At this point, it appears that popups generally work in Liferay.
Marking as FIXED for 1.7 DR#1
icefaces-test-popup-portlet.war