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

              Ascending order - Click to sort in descending order
              Carmen Cristurean created issue -
              Carmen Cristurean made changes -
              Field Original Value New Value
              Fix Version/s EE-4.1.0.GA [ 12171 ]
              Carmen Cristurean made changes -
              Fix Version/s EE-3.3.0.GA_P04 [ 12270 ]
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Assignee Priority P2 [ 10011 ]
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Hide
              Permalink
              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.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #47608 Thu Feb 11 13:35:28 MST 2016 art.zambrano ICE-10942 corrected selector to unregister document click handler
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/wijmo/wijmo.js
              Hide
              Permalink
              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).
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              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
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #47785 Mon Feb 15 14:41:56 MST 2016 art.zambrano ICE-10942 added call to unregister the document click handler from within the hideAllMenus() function
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/wijmo/wijmo.js
              Hide
              Permalink
              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).
              Ken Fyten made changes -
              Fix Version/s EE-4.1.0.BETA [ 13072 ]
              Ken Fyten made changes -
              Fix Version/s 4.2.BETA [ 13091 ]
              Fix Version/s 4.2 [ 12870 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: