[Lens] XY chart -long legend overflows chart in editor Feature:Lens (e…

…lastic#70702)
mbondyra · Jul 14, 2020 · aaf0895 · aaf0895
1 parent 17d8e86
commit aaf0895
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 23 deletions.
diff --git a/x-pack/plugins/lens/public/editor_frame_service/editor_frame/_workspace_panel_wrapper.scss b/x-pack/plugins/lens/public/editor_frame_service/editor_frame/_workspace_panel_wrapper.scss
@@ -36,3 +36,7 @@
     }
   }
 }
+
+.lnsWorkspacePanelWrapper__toolbar {
+  margin-bottom: $euiSizeS;
+}
diff --git a/...lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel_wrapper.tsx b/...lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel_wrapper.tsx
@@ -66,8 +66,8 @@ export function WorkspacePanelWrapper({
     [dispatch]
   );
   return (
-    <EuiFlexGroup gutterSize="s" direction="column" alignItems="stretch" responsive={false}>
-      <EuiFlexItem grow={false}>
+    <>
+      <div className="lnsWorkspacePanelWrapper__toolbar">
         <EuiFlexGroup gutterSize="s" direction="row" responsive={false}>
           <EuiFlexItem grow={false}>
             <ChartSwitch
@@ -94,26 +94,24 @@ export function WorkspacePanelWrapper({
             </EuiFlexItem>
           )}
         </EuiFlexGroup>
-      </EuiFlexItem>
-      <EuiFlexItem>
-        <EuiPageContent className="lnsWorkspacePanelWrapper">
-          {(!emptyExpression || title) && (
-            <EuiPageContentHeader
-              className={classNames('lnsWorkspacePanelWrapper__pageContentHeader', {
-                'lnsWorkspacePanelWrapper__pageContentHeader--unsaved': !title,
-              })}
-            >
-              <span data-test-subj="lns_ChartTitle">
-                {title ||
-                  i18n.translate('xpack.lens.chartTitle.unsaved', { defaultMessage: 'Unsaved' })}
-              </span>
-            </EuiPageContentHeader>
-          )}
-          <EuiPageContentBody className="lnsWorkspacePanelWrapper__pageContentBody">
-            {children}
-          </EuiPageContentBody>
-        </EuiPageContent>
-      </EuiFlexItem>
-    </EuiFlexGroup>
+      </div>
+      <EuiPageContent className="lnsWorkspacePanelWrapper">
+        {(!emptyExpression || title) && (
+          <EuiPageContentHeader
+            className={classNames('lnsWorkspacePanelWrapper__pageContentHeader', {
+              'lnsWorkspacePanelWrapper__pageContentHeader--unsaved': !title,
+            })}
+          >
+            <span data-test-subj="lns_ChartTitle">
+              {title ||
+                i18n.translate('xpack.lens.chartTitle.unsaved', { defaultMessage: 'Unsaved' })}
+            </span>
+          </EuiPageContentHeader>
+        )}
+        <EuiPageContentBody className="lnsWorkspacePanelWrapper__pageContentBody">
+          {children}
+        </EuiPageContentBody>
+      </EuiPageContent>
+    </>
   );
 }