ICEfaces
  1. ICEfaces
  2. ICE-9222

ace:menuButton > div is rendered in top left corner when disabled=true

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2, 3.3, 4.0.BETA
    • Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces3 trunk revision# 34552
    • Assignee Priority:
      P2

      Description

      When rendering a menuButton with disabled=true there is a div rendered in the top left corner. The div prevents the user from interacting with the page (in this example, I am unable to click the checkboxes).

      To reproduce:
      1) Build / deploy test app located at: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/menuButton
      2) Navigate to 'MenuButton Dynamic Attribute Test'
      3) Click rendered to display the menuButton
      4) Click disabled to disable the menuButton
      Div is rendered when the menuButton is disabled.

        Activity

        Cruz Miraback created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Arturo Zambrano [ artzambrano ]
        Priority Major [ 3 ] Minor [ 4 ]
        Assignee Priority P2 [ 10011 ]
        Cruz Miraback made changes -
        Attachment screenshot-1.jpg [ 15841 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #34600 Tue Apr 30 10:04:50 MDT 2013 art.zambrano ICE-9222 prevented 'disabled' setting from being passed to the wijmo menu init function, since it's not necessary
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/menu/menu.js
        Hide
        Arturo Zambrano added a comment -

        Committed fox at revision 34600.

        The wijmo menu widget has its own way to disable menus by creating a transparent div over the menu to prevent interaction. The script to do this copies the menu's coordinates and dimensions. In the case of the MenuButton, the menu is already hidden, and only the button itself needs to be disabled. So, we simply avoid passing the 'disabled' parameter to wijmo, since the menus are already hidden and the user can't interact with them.

        Show
        Arturo Zambrano added a comment - Committed fox at revision 34600. The wijmo menu widget has its own way to disable menus by creating a transparent div over the menu to prevent interaction. The script to do this copies the menu's coordinates and dimensions. In the case of the MenuButton, the menu is already hidden, and only the button itself needs to be disabled. So, we simply avoid passing the 'disabled' parameter to wijmo, since the menus are already hidden and the user can't interact with them.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s EE-3.3.0.GA [ 10572 ]
        Ken Fyten made changes -
        Fix Version/s 4.0 [ 11382 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: