ICEfaces
  1. ICEfaces
  2. ICE-9879

ace:contextMenu - Menu doesn't disappear on other right clicks

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P01
    • Fix Version/s: EE-3.3.0.GA_P02, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      IE, Chrome
    • Assignee Priority:
      P2

      Description

      After displaying the ace:contextMenu, any subsequent right clicks outside of the element that shows the menu, do not remove the menu from the page. This is seen on IE and Chrome browsers. This is not seen in Firefox.

      Also if there are two contextMenu components on the page, its possible to show both of these at the same time. This can be seen with the Showcase -> Context Menu -> Per Component Demo.

        Activity

        Arran Mccullough created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Arturo Zambrano [ artzambrano ]
        Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
        Fix Version/s 4.0 [ 11382 ]
        Affects Version/s EE-3.3.0.GA_P02 [ 11371 ]
        Priority Major [ 3 ] Minor [ 4 ]
        Assignee Priority P2 [ 10011 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #40699 Thu Apr 03 11:53:35 MDT 2014 art.zambrano ICE-9879 added oncontextmenu event trigger to document click handler in wijmo code so menus are hidden on right clicks as well
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/wijmo/wijmo.js
        Hide
        Arturo Zambrano added a comment -

        Committed fix to 4.0 trunk at revision 40699 and to 3.3 EE branch at revision 40700. Added oncontextmenu event trigger to document click handler in wijmo code, so menus are hidden on right clicks as well.

        Testing notes: to reproduce/test you can modify the "ace:contextMenu - Table Integration Example" demo. Instead of a single context menu for the whole table, move the context menu component inside a column in the table to trigger for another component inside the same column/row, such as the h:outputText component in that table, so that there's a context menu per row. With this fix, when triggering a context menu in one row, the previous open menu should disappear.

        Show
        Arturo Zambrano added a comment - Committed fix to 4.0 trunk at revision 40699 and to 3.3 EE branch at revision 40700. Added oncontextmenu event trigger to document click handler in wijmo code, so menus are hidden on right clicks as well. Testing notes: to reproduce/test you can modify the "ace:contextMenu - Table Integration Example" demo. Instead of a single context menu for the whole table, move the context menu component inside a column in the table to trigger for another component inside the same column/row, such as the h:outputText component in that table, so that there's a context menu per row. With this fix, when triggering a context menu in one row, the previous open menu should disappear.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Liana Munroe added a comment -

        Confirmed fixed using the suggested test implementation and tested in current showcase, Icefaces ee-3.3.0 maintenance branch r40704 and Icefaces 4 trunk r40704
        All browsers, Tomcat 7.

        Show
        Liana Munroe added a comment - Confirmed fixed using the suggested test implementation and tested in current showcase, Icefaces ee-3.3.0 maintenance branch r40704 and Icefaces 4 trunk r40704 All browsers, Tomcat 7.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #40798 Fri Apr 11 14:45:55 MDT 2014 art.zambrano ICE-9943, ICE-9879 removed namespace from click event registration, so that the event can be triggered multiple times
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/wijmo/wijmo.js
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: