[incubator-superset] branch omnibar updated: adressed comments

[hugh]

This is an automated email from the ASF dual-hosted git repository.

hugh pushed a commit to branch omnibar
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/omnibar by this push:
     new df231ba  adressed comments
df231ba is described below

commit df231bac27c56509f9847730480238bc9e7c8dd3
Author: hughhhh <hmi...@lyft.com>
AuthorDate: Thu Jan 31 15:03:46 2019 -0800

    adressed comments
---
 superset/assets/src/components/OmniContainer.jsx   | 56 ++++++++++++++++++++--
 .../assets/src/dashboard/components/Dashboard.jsx  |  7 ++-
 2 files changed, 58 insertions(+), 5 deletions(-)

diff --git a/superset/assets/src/components/OmniContainer.jsx 
b/superset/assets/src/components/OmniContainer.jsx
index acaae69..0554ad9 100644
--- a/superset/assets/src/components/OmniContainer.jsx
+++ b/superset/assets/src/components/OmniContainer.jsx
@@ -18,10 +18,20 @@
  */
 import React from 'react';
 import { Modal } from 'react-bootstrap';
+import PropTypes from 'prop-types';
 import { t } from '@superset-ui/translation';
 import { SupersetClient } from '@superset-ui/connection';
-
 import Omnibar from 'omnibar';
+import {
+  Logger,
+  ActionLog,
+  LOG_ACTIONS_OMNIBAR_TRIGGERED,
+} from '../logger';
+
+const propTypes = {
+  impressionId: PropTypes.string.isRequired,
+  dashboardId: PropTypes.number.isRequired,
+};
 
 const getDashboards = query =>
   // todo: Build a dedicated endpoint for dashboard searching
@@ -38,18 +48,56 @@ const getDashboards = query =>
             title: t('An error occurred while fethching Dashboards'),
         }));
 
-export default class OmniContainer extends React.Component {
+class OmniContainer extends React.Component {
   constructor(props) {
     super(props);
     this.state = {
-      showOmni: true,
+      showOmni: false,
     };
+    this.handleKeydown = this.handleKeydown.bind(this);
+  }
+
+  componentDidMount() {
+    document.addEventListener('keydown', this.handleKeydown);
   }
+
+  componentWillUnmount() {
+    document.removeEventListener('keydown', this.handleKeydown);
+  }
+
+  handleKeydown(event) {
+    const controlOrCommand = event.ctrlKey || event.metaKey;
+    if (controlOrCommand) {
+      const isK = event.key === 'k' || event.keyCode === 83;
+      if (isK) {
+        this.setState({ showOmni: !this.state.showOmni });
+
+        // Get first input in the modal div
+        document
+          .getElementsByClassName('modal-dialog')[0]
+          .getElementsByTagName('input')[0]
+          .focus();
+
+        Logger.send(
+          new ActionLog({
+            impressionId: this.props.impressionId, // impo
+            source: 'dashboard',
+            sourceId: this.props.dashboardId, // sourceId: 
this.props.dashboardId
+            eventNames: LOG_ACTIONS_OMNIBAR_TRIGGERED,
+          }),
+        );
+      }
+    }
+  }
+
   render() {
       return (
         <Modal show={this.state.showOmni} ref={this.exampleRef}>
           <Omnibar placeholder="Search for dashboards.." 
extensions={[getDashboards]} />
         </Modal>
-      )
+      );
   }
 }
+
+OmniContainer.propTypes = propTypes;
+export default OmniContainer;
diff --git a/superset/assets/src/dashboard/components/Dashboard.jsx 
b/superset/assets/src/dashboard/components/Dashboard.jsx
index 6394793..57cc661 100644
--- a/superset/assets/src/dashboard/components/Dashboard.jsx
+++ b/superset/assets/src/dashboard/components/Dashboard.jsx
@@ -266,9 +266,14 @@ class Dashboard extends React.PureComponent {
   }
 
   render() {
+    const {
+      impressionId,
+      dashboardInfo: { id },
+    } = this.props;
+
     return (
       <React.Fragment>
-        <OmniContianer />
+        <OmniContianer impressionId={impressionId} dashboardId={id} />
         <DashboardBuilder />
       </React.Fragment>
     );