Merge pull request #2 from monfera/cqliu1-canvas/help-menu

Fix: reintroduce additional call on keyboard event
cqliu1 · Feb 21, 2019 · f6b0c13 · f6b0c13
2 parents 7b2fa27 + daaa871
commit f6b0c13
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/x-pack/plugins/canvas/public/components/workpad_page/event_handlers.js b/x-pack/plugins/canvas/public/components/workpad_page/event_handlers.js
@@ -127,6 +127,7 @@ const handleMouseDown = (commit, e, isEditable) => {
 export const eventHandlers = {
   onMouseDown: props => e => handleMouseDown(props.commit, e, props.isEditable),
   onMouseMove: props => e => handleMouseMove(props.commit, e, props.isEditable),
+  onKeyDown: props => () => props.commit('keyboardEvent'), // dummy event
   onWheel: props => e => handleWheel(props.commit, e, props.isEditable),
   resetHandler: () => () => resetHandler(),
 };
diff --git a/x-pack/plugins/canvas/public/components/workpad_page/workpad_page.js b/x-pack/plugins/canvas/public/components/workpad_page/workpad_page.js
@@ -39,6 +39,7 @@ export class WorkpadPage extends PureComponent {
     width: PropTypes.number.isRequired,
     isEditable: PropTypes.bool.isRequired,
     onDoubleClick: PropTypes.func,
+    onKeyDown: PropTypes.func,
     onMouseDown: PropTypes.func,
     onMouseMove: PropTypes.func,
     onMouseUp: PropTypes.func,
@@ -70,6 +71,7 @@ export class WorkpadPage extends PureComponent {
       width,
       isEditable,
       onDoubleClick,
+      onKeyDown,
       onMouseDown,
       onMouseMove,
       onMouseUp,
@@ -143,6 +145,7 @@ export class WorkpadPage extends PureComponent {
           width,
           cursor,
         }}
+        onKeyDown={onKeyDown}
         onMouseMove={onMouseMove}
         onMouseUp={onMouseUp}
         onMouseDown={onMouseDown}