ICEfaces
  1. ICEfaces
  2. ICE-9609

ace:menu > Styling issues when using sliding menu type

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA
    • Fix Version/s: 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces3 trunk revision# 38279
      All browsers
    • Assignee Priority:
      P3

      Description

      Test app located at: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/menu

      1) When using sliding menu type and the maxHeight is set so that the whole menu is not displayed, the vertical scrollbar only renders after navigating into one of the subMenus and navigating back. On initial render the scrollbar is not displayed so there is no way of seeing the other menuItems.

      To reproduce:
      1. Build / deploy test app
      2. Navigate to 'Menu Dynamic Attribute Test'
      3. Select type 'sliding'
      4. Select maxHeight '90'
      5. Select 'rendered' (Notice no scrollbar is displayed on the menu)
      6. Click one of the subMenus
      7. Click 'Back' (Scrollbar is now displayed)

      2) When using sliding menu type in combination with the style attribute the menu renders with a height of 0. This appears to be because the custom styling is overriding the default height value (and z-index) in the second inner div (span>div>div). From what I can tell, the styling is being rendered on the wrong div when using this menu type, and it should instead be rendered on the first inner div (span>div) like it does for the other menu types (plain, tiered).

      To reproduce:
      1. Build / deploy test app
      2. Navigate to 'Menu Attribute Test'
      3. Scroll down to test with title 'Sliding Menu w/ style=border:1px red solid;' (Notice how the menu is displayed)

      3) Related to #2, when using sliding menu type and the styleClass attribute the custom class renders on the second inner div instead of the first, where as the other menu types render on the first.

      To reproduce:
      1. Build / deploy test app
      2. Navigate to 'Menu Attribute Test'
      3. Scroll down to test with title 'Sliding Menu w/ styleClass=blueBorderThick' (Notice that the border is displayed in the 2nd inner div)

      4) Related to #2, when using sliding menu type the z-index attribute isn't functioning. This is for the same reason as the above two issues, the z-index attribute is rendered on the wrong div (should be first inner div instead of second).

      To reproduce:
      1. Build / deploy test app
      2. Navigate to 'Menu Dynamic Attribute Test'
      3. Select type 'sliding'
      4. Select position 'dynamic'
      5) Select 'rendered'
      6) Click the gray box to display the menu component (notice the menu is rendered below the divs even though it should have a higher z-index)

        Activity

        Cruz Miraback created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Arran Mccullough [ arran.mccullough ]
        Priority Major [ 3 ] Minor [ 4 ]
        Assignee Priority P2 [ 10011 ]
        Ken Fyten made changes -
        Fix Version/s 4.0 [ 11382 ]
        Fix Version/s 4.0.BETA [ 10770 ]
        Ken Fyten made changes -
        Assignee Arran Mccullough [ arran.mccullough ] Arturo Zambrano [ artzambrano ]
        Ken Fyten made changes -
        Assignee Priority P2 [ 10011 ] P3 [ 10012 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #42041 Wed Jul 30 14:48:40 MDT 2014 art.zambrano ICE-9609 fix to refresh menu right after creation so that all styling can be applied correctly after the menu is finally made visible
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/menu/menu.js
        Hide
        Arturo Zambrano added a comment -

        r42041: fix to refresh menu right after creation so that all stylings can be applied correctly after the menu is finally made visible.

        Show
        Arturo Zambrano added a comment - r42041: fix to refresh menu right after creation so that all stylings can be applied correctly after the menu is finally made visible.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Liana Munroe added a comment -

        Verified fixed with Icefaces 4 trunk r42046 with the exception of issue #4. Reopening for this reason. Please see attached screen zIndex.PNG

        Show
        Liana Munroe added a comment - Verified fixed with Icefaces 4 trunk r42046 with the exception of issue #4. Reopening for this reason. Please see attached screen zIndex.PNG
        Liana Munroe made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Liana Munroe made changes -
        Attachment zIndex.PNG [ 17301 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #42053 Thu Jul 31 14:41:02 MDT 2014 art.zambrano ICE-9609 fix to correctly apply the zIndex setting to sliding menus
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/menu/menu.js
        Hide
        Arturo Zambrano added a comment -

        r42053: committed fix to correctly apply the zIndex setting to sliding menus.

        Show
        Arturo Zambrano added a comment - r42053: committed fix to correctly apply the zIndex setting to sliding menus.
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Liana Munroe added a comment -

        Verified issue 4 now resolved. Icefaces 4 trunk r42054, all browsers.

        Show
        Liana Munroe added a comment - Verified issue 4 now resolved. Icefaces 4 trunk r42054, all browsers.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Cruz Miraback
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: