Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 4.2
-
Component/s: ACE-Components, QA
-
Labels:None
-
Environment:ICEfaces4 trunk r.49164
Browsers: all
Server: Tomcat7.0.42
-
Assignee Priority:P2
Description
QA test applications:
http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/menuBar
http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/menu
Unable to render the icon image when getting its value from the bean, or when entering a static value for image path on the .xhtml page. This may be an application issue, I just could not get it to work.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
ICEfaces4 trunk r.49175/ IE11, Chrome53, FF47: updated the QA test apps for menu and menuBar, and they both display the icons now.
However the icon position changes in case of the ace:menuBar component: the icon renders initially before the text value, and moves after the text value when changing dynamically any of the attributes of the menuItem. This behavior is not seen on the menuItem within the ace:menu component.
The position of the icon changes also for the menuItem within a contextMenu component.
r49217: added node with 'wijmo-wijmenu-text' styleClass to always display icons correctly across individual menuItem updates
The inner html of the anchor elements of ace:menuItem is dynamically wrapped in the client by a node with the 'wijmo-wijmenu-text' style class, which makes the text and the icons look as they are. After individually updating a menuItem, this node was lost. So, this fix consists in always rendering this node from the Renderer class to keep looks consistent.
Verified ICEfaces4 trunk r.49219 on IE8/11, MSEdge38, FF47, Chrome53/ Tomcat7.0.42.
r49175: corrected the TLD documentation for the ace:menuItem icon attribute, matching the documentation for the ace:submenu icon attribute
This was a TLD documentation error that had gone unnoticed since the creation of this component. The value of the icon attribute should be a CSS class name or class names to display an icon. This is the way it works in the ace:submenu component. The ace:menuBar demo in the showcase app shows its correct usage for both, ace:submenu and ace:menuItem.