ICEfaces
  1. ICEfaces
  2. ICE-2213

Portlet menu styling

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.7DR#1
          • Fix Version/s: 1.7DR#3, 1.7
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All

            Description

            I don't think that our portlet menu CSS style classes are used properly. We differentiate between classes for top level menus, and submenus, whereas they differentiate between menuitems that are leafs versus menuitems that lead to submenus.

            http://www.ja-sig.org/wiki/display/UPC/JSR-168+PLT.C+CSS+Style+Definitions+section


            public class com.icesoft.faces.component.PORTLET_CSS_DEFAULT {
                                  //PLT.C.6 Menus
              public static final String PORTLET_MENU = "portlet-menu"; // (menuBar)
              public static final String PORTLET_MENU_ITEM = "portlet-menu-item"; // (root level menuItem)
              public static final String PORTLET_MENU_ITEM_HOVER = "portlet-menu-item-hover"; // (root level menuItem:hover)
              public static final String PORTLET_MENU_CASCADE_ITEM = "portlet-menu-cascade-item"; // (submenu menuItem)
            }

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              yip.ng added a comment -

              How to test:

              In core\src\com\icesoft\faces\util\CoreUtils.java

              in the method addPortletStyleClassToQualifiedClass(String qualifiedStyleClass, String defaultClass, String portletClass, boolean disabled)

              change the if condition to be always true:
              // if (isPortletEnvironment() && isRenderPortletStyleClass()) {
              if (true) {

              In component showcase, go to the menu bar example. Use Firebug to inspect the style classes as you select or hover over a menu item. (See the attached screenshot.)

              Show
              yip.ng added a comment - How to test: In core\src\com\icesoft\faces\util\CoreUtils.java in the method addPortletStyleClassToQualifiedClass(String qualifiedStyleClass, String defaultClass, String portletClass, boolean disabled) change the if condition to be always true: // if (isPortletEnvironment() && isRenderPortletStyleClass()) { if (true) { In component showcase, go to the menu bar example. Use Firebug to inspect the style classes as you select or hover over a menu item. (See the attached screenshot.)
              Hide
              Permalink
              yip.ng added a comment -

              Screenshot of test results.

              Show
              yip.ng added a comment - Screenshot of test results.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Mark Collette
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: