ICEfaces
  1. ICEfaces
  2. ICE-5289

Cannot Apply Style to Top Level MenuItem

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8.2, 1.8.2-EE-GA
    • Fix Version/s: 1.8.2-EE-GA_P01, 1.8.3
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      N/A
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.)
    • Workaround Exists:
      Yes
    • Workaround Description:
      Create a style in the stylesheet, add this style in the styleClass attribute of the menuItem tag. However, this class name has to have the String "Item" added as a suffix if it is a top level menuItem.

      Description

      Client tried to apply a style to a top level menItem tag using the style attribute. The style attribute is not documented in the TLD for this tag and is not passed through to the source.

      The other option is to use the styleClass attribute as outlined in the workaround. However, for the top level menuItem, the String "Item" is being added as a suffix to the class when rendered to the browser and this is not documented in the TLD.

        Activity

        Hide
        Mark Collette added a comment - - edited

        Added the style attribute to the ice:menuItem component, and made it render that out.

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

        icefaces-ee-1.8.2
        svn merge -r20756:20757 http://server.ice:8888/svn/ossrepo/icefaces/trunk/icefaces/
        Subversion 20758
        icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menunode-props.xml
        icefaces\component\src\com\icesoft\faces\component\menubar\MenuItem.java
        icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java

        Show
        Mark Collette added a comment - - edited Added the style attribute to the ice:menuItem component, and made it render that out. TRUNK Subversion 20757 icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menunode-props.xml icefaces\component\src\com\icesoft\faces\component\menubar\MenuItem.java icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java icefaces-ee-1.8.2 svn merge -r20756:20757 http://server.ice:8888/svn/ossrepo/icefaces/trunk/icefaces/ Subversion 20758 icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menunode-props.xml icefaces\component\src\com\icesoft\faces\component\menubar\MenuItem.java icefaces\component\src\com\icesoft\faces\component\menubar\MenuItemRenderer.java
        Hide
        Joanne Bai added a comment -

        Test app is added to repo\qa\trunk\Regression\ICE-5289

        Verified it on ICEfaces-ee-1.8.2 revision 20758 using tomcat 6 + FF3.5 and IE8

        • The style attribute is rendered correctly
        • But the menu items do not show in correct position if the browser window is shrunk and not in its maximum size (the menu items kind of hide themselves and stay left to the menu bar)
        Show
        Joanne Bai added a comment - Test app is added to repo\qa\trunk\Regression\ ICE-5289 Verified it on ICEfaces-ee-1.8.2 revision 20758 using tomcat 6 + FF3.5 and IE8 The style attribute is rendered correctly But the menu items do not show in correct position if the browser window is shrunk and not in its maximum size (the menu items kind of hide themselves and stay left to the menu bar)
        Hide
        Joanne Bai added a comment -

        New JIRAs ICE-5428 and ICE-5429 are created to address the menu bar issues found while verifying this jira. Thanks to Mark!

        Show
        Joanne Bai added a comment - New JIRAs ICE-5428 and ICE-5429 are created to address the menu bar issues found while verifying this jira. Thanks to Mark!

          People

          • Assignee:
            Joanne Bai
            Reporter:
            Brad Kroeger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: