ICEfaces
  1. ICEfaces
  2. ICE-11133

ace:menuItem - icon image fails to render

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.2
          • Fix Version/s: 4.2.BETA, 4.2
          • Component/s: ACE-Components, QA
          • Labels:
            None
          • Environment:
            ICEfaces4 trunk r.49164
            Browsers: all
            Server: Tomcat7.0.42
          • Assignee Priority:
            P2

            Description

            ace:menuBar, ace:menu > /menuItemDynamicUpdate.jsf
            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

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified ICEfaces4 trunk r.49219 on IE8/11, MSEdge38, FF47, Chrome53/ Tomcat7.0.42.

              Show
              Carmen Cristurean added a comment - Verified ICEfaces4 trunk r.49219 on IE8/11, MSEdge38, FF47, Chrome53/ Tomcat7.0.42.
              Hide
              Permalink
              Arturo Zambrano added a comment -

              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.

              Show
              Arturo Zambrano added a comment - 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.
              Hide
              Permalink
              Carmen Cristurean added a comment - - edited

              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.

              Show
              Carmen Cristurean added a comment - - edited 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.
              Hide
              Permalink
              Arturo Zambrano added a comment -

              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.

              Show
              Arturo Zambrano added a comment - 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.

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Carmen Cristurean
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: