Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: EE-4.1.0.GA, EE-3.3.0.GA_P04
-
Fix Version/s: 4.2.BETA, 4.2, EE-3.3.0.GA_P05
-
Component/s: ACE-Components
-
Labels:None
-
Environment:All
-
Assignee Priority:P1
-
Support Case References:Support Case #13859 - https://icesoft.my.salesforce.com/5007000001drgdk
Description
This seems to be an issue for both using the menu modal and having the menu structure on the page.
-
Hide
- Case13548Example5.war
- 13.76 MB
- Arran Mccullough
-
- META-INF/MANIFEST.MF 0.1 kB
- META-INF/context.xml 0.1 kB
- WEB-INF/classes/CountryBean$1.class 1 kB
- WEB-INF/classes/CountryBean$2.class 1 kB
- WEB-INF/classes/CountryBean.class 5 kB
- WEB-INF/.../AbstractPushGroupNameGenerator.class 1 kB
- WEB-INF/classes/.../AttachmentWrapper.class 4 kB
- WEB-INF/classes/.../IUploadableObject.class 0.2 kB
- WEB-INF/.../RenderableController.class 3 kB
- WEB-INF/classes/util/SelectionList.class 6 kB
- WEB-INF/classes/.../SelectionWrapper.class 2 kB
- WEB-INF/.../UploadController$InvalidFileStatus.class 1 kB
- WEB-INF/.../UploadController$UploadSuccessStatus.class 1 kB
- WEB-INF/classes/.../UploadController.class 10 kB
- WEB-INF/classes/util/UploadHandler.class 7 kB
- WEB-INF/lib/commons-beanutils.jar 226 kB
- WEB-INF/lib/commons-digester.jar 140 kB
- WEB-INF/lib/commons-logging.jar 52 kB
- WEB-INF/lib/icefaces-ace.jar 5.75 MB
- WEB-INF/lib/icefaces-compat.jar 4.18 MB
- WEB-INF/lib/icefaces.jar 640 kB
- WEB-INF/lib/icepush.jar 207 kB
- WEB-INF/lib/javax.faces.jar 2.55 MB
- WEB-INF/web.xml 2 kB
- index.xhtml 0.5 kB
- welcomeICEfaces.xhtml 1 kB
-
- Case13548Example5.zip
- 29 kB
- Arran Mccullough
-
Hide
- Case13859Example2.war
- 9.92 MB
- Arran Mccullough
-
- META-INF/MANIFEST.MF 0.1 kB
- META-INF/context.xml 0.1 kB
- WEB-INF/classes/com/.../support/Item.class 0.3 kB
- WEB-INF/classes/.../support/TestBean.class 1.0 kB
- WEB-INF/lib/icefaces-ace.jar 6.24 MB
- WEB-INF/lib/icefaces.jar 693 kB
- WEB-INF/lib/javax.faces-2.2.13.jar 2.99 MB
- WEB-INF/web.xml 2 kB
- index.xhtml 0.5 kB
- welcomeICEfaces.xhtml 2 kB
-
Hide
- Case13859Example2.zip
- 20 kB
- Arran Mccullough
-
- Case13859Example2/build.xml 3 kB
- Case13859Example2/.../ant-deploy.xml 2 kB
- Case13859Example2/.../build-impl.xml 80 kB
- Case13859Example2/.../genfiles.properties 0.5 kB
- Case13859Example2/.../private.properties 0.6 kB
- Case13859Example2/.../project.properties 4 kB
- Case13859Example2/nbproject/project.xml 0.9 kB
- Case13859Example2/src/conf/MANIFEST.MF 0.0 kB
- Case13859Example2/src/.../support/Item.java 0.1 kB
- Case13859Example2/src/.../TestBean.java 0.8 kB
- Case13859Example2/web/index.xhtml 0.5 kB
- Case13859Example2/web/.../context.xml 0.1 kB
- Case13859Example2/web/WEB-INF/web.xml 2 kB
- Case13859Example2/.../welcomeICEfaces.xhtml 2 kB
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Attached test case that shows this issue.
Steps:
- Load welcomeICEfaces.jsf
- Hover over the "One" menuBar, the sub menu items are shown.
- In the drop down change from "One" to "Two". The menuBar is updated but loses its styling and no longer shows the sub menu's.
rev 49150 -> modified menuBar script to have hashcode which consists of submenu labels.
Tried forceUpdateCounter, but that only updates the menuBar node and does not update the script (gets wiped out by domdiff), so needed to use another field to save the labels of a submenu to store it in the menuBar. Tested with a single subMenu, but this will not fix if multiple subMenus, so a little more work to do.
rev 49152 for ee3 maintenance branch. fix for multiple submenu children of menu component and using hashcode to make writer fn more performant.
rev 49154 for IF 3.3 maintenance and 49153 for IF4.
QA should test with multiple ace:submenu component with dynamic labels (the test case for this one just has a single submenu). When adding dynamic tests for child components (that depend on parent script for updating), multiple dynamic values should be used as well.
Verified with attached test case IF4 trunk r.49159/EE-3.3.0-maintenance branch r.49154 on IE8/IE11, FF47, Chrome52, Tomcat7.0.42.
IF4 trunk r.49164/EE-3.3.0-maintenance branch r.49154/all browsers:
This issue can also be reproduced with ace:menu, if using "type=tiered" or "type=sliding"; after changing ace:submenu label, the ace:submenu loses its style and the menu items can no longer be rendered.
QA test app:
http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/menu
Test page: /submenuDynamicUpdate.jsf
It also can be seen with ace:contextMenu:
http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/contextMenu
Test page: /submenuDynamicUpdate.jsf
ICE-11136 and ICE-11137 have been created for ace:menu, ace:contextMenu.
Customer has found a regression related to these fixes. When a menu is displayed via a hover, the menu is closed when an ICEpush update is made on the page. This is causing major usability issues with the menu component.
Attached test case and code to reproduce the regression.
Steps:
- Load welcomeICEfaces.jsf
- Hover over the Menu bar.
- On each push update, the menu is collapsed.
reset submentLabels to empty string each time the menuBar is encoded so push does not compound the same value again and again (causing the script to re-evaluate causing a reset on the menuBar).
rev 49540 for IF3.3 mtce, rev 49541 for IF4 and QA rev 46017 to add simplified test.
Verified ICEfaces ee-3.3.0 maintenance branch r49541 using attached test case, IE 11, FF 47, Chrome 53. Regression tested showcase and QA menuBar test apps using FF 47, no issues detected.
Verified ICEfaces e4 trunk r49541 using new test case in QA app test case /sc13859WithPush.jsf, IE 11, FF 47, Chrome 53. Regression tested showcase and QA menuBar test apps using FF 47, no issues detected.
This seems to be similar to
ICE-11121, it might be worthwhile to check other similar components for the same issue.