ICEfaces
  1. ICEfaces
  2. ICE-2745

onclick missing on menuItem

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 1.6.2, 1.7Beta1
          • Fix Version/s: 1.7RC1, 1.7
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            Any
          • Affects:
            Sample App./Tutorial

            Description

            The onclick attribute does not work in conjunction with ice:menuItem. Even if added at a page level, it doesn't seem to render on the page or be passed through to the component.

            In this specific case, we cannot add Loopfuse javascript based tracking to menuItem links.

              Activity

              Hide
              Permalink
              Mark Collette added a comment -

              I added an "onclick" attribute to MenuItem, and made the MenuItemRenderer pass that on to the generated anchor tag.

              Subversion 15929
              icefaces\component\src\com\icesoft\faces\component\menubar\MenuItem.java
              icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java
              icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menunode-props.xml

              Show
              Mark Collette added a comment - I added an "onclick" attribute to MenuItem, and made the MenuItemRenderer pass that on to the generated anchor tag. Subversion 15929 icefaces\component\src\com\icesoft\faces\component\menubar\MenuItem.java icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menunode-props.xml

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Carlo Guglielmin
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: