Browse Source

wip: changes

Torkel Ödegaard 7 years ago
parent
commit
666e8e8330

+ 7 - 3
public/app/features/dashboard/dashgrid/EditorTabBody.tsx

@@ -5,7 +5,7 @@ import { FadeIn } from 'app/core/components/Animations/FadeIn';
 interface Props {
   children: JSX.Element;
   heading: string;
-  renderToolbar?: () => JSX.Element | JSX.Element[];
+  renderToolbar?: () => JSX.Element;
   toolbarItems?: EditorToolBarView[];
 }
 
@@ -106,8 +106,12 @@ export class EditorTabBody extends PureComponent<Props, State> {
         <div className="toolbar">
           <div className="toolbar__heading">{heading}</div>
           {renderToolbar && renderToolbar()}
-          <div className="gf-form--grow" />
-          {toolbarItems.map(item => this.renderButton(item))}
+          {toolbarItems.length > 0 && (
+            <>
+              <div className="gf-form--grow" />
+              {toolbarItems.map(item => this.renderButton(item))}
+            </>
+          )}
         </div>
         <div className="panel-editor__scroll">
           <CustomScrollbar autoHide={false}>

+ 10 - 3
public/app/features/dashboard/dashgrid/VisualizationTab.tsx

@@ -169,6 +169,10 @@ export class VisualizationTab extends PureComponent<Props, State> {
             />
             <i className="gf-form-input-icon fa fa-search" />
           </label>
+          <div className="flex-grow" />
+          <button className="btn btn-link" onClick={this.onCloseVizPicker}>
+            <i className="fa fa-chevron-up" />
+          </button>
         </>
       );
     } else {
@@ -183,14 +187,17 @@ export class VisualizationTab extends PureComponent<Props, State> {
   };
 
   onTypeChanged = (plugin: PanelPlugin) => {
-    // this.setState({ isVizPickerOpen: false });
-    this.props.onTypeChanged(plugin);
+    if (plugin.id === this.props.plugin.id) {
+      this.setState({ isVizPickerOpen: false });
+    } else {
+      this.props.onTypeChanged(plugin);
+    }
   };
 
   render() {
     const { plugin } = this.props;
     const { isVizPickerOpen, searchQuery } = this.state;
-    const toolbarItems: EditorToolBarView = [];
+    const toolbarItems: EditorToolBarView[] = [];
 
     if (!isVizPickerOpen) {
       toolbarItems.push({

+ 0 - 1
public/app/features/dashboard/dashgrid/VizTypePicker.tsx

@@ -62,7 +62,6 @@ export class VizTypePicker extends PureComponent<Props> {
   };
 
   render() {
-    const { onClose } = this.props;
     const filteredPluginList = this.getFilteredPluginList();
 
     return (

+ 3 - 3
public/sass/_variables.dark.scss

@@ -271,7 +271,7 @@ $menu-dropdown-shadow: 5px 5px 20px -5px $black;
 $tab-border-color: $dark-4;
 
 // Toolbar
-$toolbar-bg: $input-bg;
+$toolbar-bg: $input-black;
 
 // Pagination
 // -------------------------
@@ -377,12 +377,12 @@ $checkbox-color: $dark-1;
 $panel-editor-shadow: 0 0 20px black;
 $panel-editor-border: 1px solid $dark-3;
 $panel-editor-side-menu-shadow: drop-shadow(0 0 10px $black);
-$panel-editor-toolbar-view-bg: $black;
+$panel-editor-toolbar-view-bg: $input-black;
 $panel-editor-viz-item-shadow: 0 0 8px $dark-5;
 $panel-editor-viz-item-border: 1px solid $dark-5;
 $panel-editor-viz-item-shadow-hover: 0 0 4px $blue;
 $panel-editor-viz-item-border-hover: 1px solid $blue;
-$panel-editor-viz-item-bg: $black;
+$panel-editor-viz-item-bg: $input-black;
 $panel-editor-tabs-line-color: #e3e3e3;
 $panel-editor-viz-item-bg-hover: darken($blue, 47%);
 $panel-editor-viz-item-bg-hover-active: darken($orange, 45%);

+ 1 - 2
public/sass/components/_panel_editor.scss

@@ -35,7 +35,6 @@
   background: $page-bg;
   margin: 0 20px 0 84px;
   border-radius: 3px;
-  border: $panel-editor-border;
   box-shadow: $panel-editor-shadow;
 }
 
@@ -134,7 +133,7 @@
 
 .viz-picker {
   background: $toolbar-bg;
-  margin: -40px -20px 40px 106px;
+  margin: -40px -20px 40px -20px;
   padding: 20px;
   position: relative;
 }

+ 4 - 0
public/sass/utils/_utils.scss

@@ -83,6 +83,10 @@ button.close {
   position: absolute;
 }
 
+.flex-grow {
+  flex-grow: 1;
+}
+
 .center-vh {
   display: flex;
   align-items: center;