From: Michal Klobukowski <michal.klobukowski@contractors.roche.com>
Date: Mon, 9 Jul 2018 12:52:01 +0000 (+0200)
Subject: Block browser context menu inside app's context menu
X-Git-Tag: 1.2.0~57^2
X-Git-Url: https://git.arvados.org/arvados-workbench2.git/commitdiff_plain/4187ee4067e67370f6708ba1a0121ecf0a1ac756

Block browser context menu inside app's context menu

Feature #13770

Arvados-DCO-1.1-Signed-off-by: Michal Klobukowski <michal.klobukowski@contractors.roche.com>
---

diff --git a/src/components/context-menu/context-menu.tsx b/src/components/context-menu/context-menu.tsx
index 7751be49..6ac1207b 100644
--- a/src/components/context-menu/context-menu.tsx
+++ b/src/components/context-menu/context-menu.tsx
@@ -27,7 +27,8 @@ export default class ContextMenu<T> extends React.PureComponent<ContextMenuProps
             open={!!anchorEl}
             onClose={onClose}
             transformOrigin={DefaultTransformOrigin}
-            anchorOrigin={DefaultTransformOrigin}>
+            anchorOrigin={DefaultTransformOrigin}
+            onContextMenu={this.handleContextMenu}>
             <List dense>
                 {actions.map((group, groupIndex) =>
                     <React.Fragment key={groupIndex}>
@@ -48,4 +49,9 @@ export default class ContextMenu<T> extends React.PureComponent<ContextMenuProps
             </List>
         </Popover>;
     }
+
+    handleContextMenu = (event: React.MouseEvent<HTMLElement>) => {
+        event.preventDefault();
+        this.props.onClose();
+    }
 }