ICEfaces
  1. ICEfaces
  2. ICE-10942

ace:menu right-click trigger event fails

    Details

    • Assignee Priority:
      P2

      Description

      The ace:menu "right-click" trigger event does not work more than once, causing the webdriver test to fail.
      This can be reproduced manually with the QA ace:menu test application, on the dynamic attribute test page:
      - go to /menuDynAttribute.jsf
      - enable "rendered" attribute, select position=dynamic.
      - under "Additional Testing For Dynamic Position" select: trigger=<any option>; triggerEvent=<rtclick>
      - right click on the selected trigger component, (div, image, input or text); the list of menuItems opens up and remains open.
      - optional step: click on one of the menu items; the list closes, the selected value is rendered on page.
      - right click again on the trigger component; the list of menu items opens up only for a second and then closes, making impossible to click on an item.

      This issue can be reproduced also with ICEfaces 4.1.0 libraries.

      QA test app:
      http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/menu

        Activity

        Hide
        Arturo Zambrano added a comment -

        r47608: corrected selector to unregister document click handler (4.0 trunk)
        r47609: committed fix to the 3.3 EE maintenance branch

        The problem was that the event handlers to hide the menu weren't being unregistered and were being accumulated.

        Show
        Arturo Zambrano added a comment - r47608: corrected selector to unregister document click handler (4.0 trunk) r47609: committed fix to the 3.3 EE maintenance branch The problem was that the event handlers to hide the menu weren't being unregistered and were being accumulated.
        Hide
        Carmen Cristurean added a comment -

        IF4 trunk r.47611/ Chrome48, IE11, FF41: this is still an issue for every second right-click (all browsers).

        Show
        Carmen Cristurean added a comment - IF4 trunk r.47611/ Chrome48, IE11, FF41: this is still an issue for every second right-click (all browsers).
        Hide
        Arturo Zambrano added a comment -

        r47785: added call to unregister the document click handler from within the hideAllMenus() function (4.0 trunk)
        r47785: committed fix to the 3.3 EE maintenance branch

        Previously I had only tested with actually clicking anywhere else on the page to hide the menu rather than clicking on a menu item. This fix covers the case of clicking on menu items

        Show
        Arturo Zambrano added a comment - r47785: added call to unregister the document click handler from within the hideAllMenus() function (4.0 trunk) r47785: committed fix to the 3.3 EE maintenance branch Previously I had only tested with actually clicking anywhere else on the page to hide the menu rather than clicking on a menu item. This fix covers the case of clicking on menu items
        Hide
        Carmen Cristurean added a comment - - edited

        Verified ICEfaces 4 trunk r.47791/ EE-3.3.0-maintenance branch r.47791 on IE11, Chrome48, FF41.
        All QA menu test app webdriver tests and selenium1 tests pass on IF4 trunk/EE-3.3.0-maintenance branch (FF41, Chrome48).

        Show
        Carmen Cristurean added a comment - - edited Verified ICEfaces 4 trunk r.47791/ EE-3.3.0-maintenance branch r.47791 on IE11, Chrome48, FF41. All QA menu test app webdriver tests and selenium1 tests pass on IF4 trunk/EE-3.3.0-maintenance branch (FF41, Chrome48).

          People

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

            Dates

            • Created:
              Updated:
              Resolved: