Merge pull request #5 from davismcphee/discover-redesign-resize-button

[Discover] Remove some whitespace around the Unified Histogram resize button
jughosta · Sep 8, 2023 · 1ec4869 · 1ec4869
2 parents b34d3bc + 5357b53
commit 1ec4869
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 14 deletions.
diff --git a/src/plugins/unified_histogram/public/layout/layout.tsx b/src/plugins/unified_histogram/public/layout/layout.tsx
@@ -275,7 +275,7 @@ export const UnifiedHistogramLayout = ({
           chart={chart}
           breakdown={breakdown}
           appendHitsCounter={appendHitsCounter}
-          appendHistogram={showFixedPanels ? <EuiSpacer size="s" /> : <EuiSpacer size="l" />}
+          appendHistogram={<EuiSpacer size="s" />}
           disableAutoFetching={disableAutoFetching}
           disableTriggers={disableTriggers}
           disabledActions={disabledActions}

diff --git a/src/plugins/unified_histogram/public/panels/panels_resizable.tsx b/src/plugins/unified_histogram/public/panels/panels_resizable.tsx
@@ -6,12 +6,7 @@
  * Side Public License, v 1.
  */
 
-import {
-  EuiResizableContainer,
-  useEuiTheme,
-  useGeneratedHtmlId,
-  useResizeObserver,
-} from '@elastic/eui';
+import { EuiResizableContainer, useGeneratedHtmlId, useResizeObserver } from '@elastic/eui';
 import type { ResizeTrigger } from '@elastic/eui/src/components/resizable_container/types';
 import { css } from '@emotion/react';
 import { isEqual, round } from 'lodash';
@@ -162,12 +157,6 @@ export const PanelsResizable = ({
     disableResizeWithPortalsHack();
   }, [disableResizeWithPortalsHack, resizeWithPortalsHackIsResizing]);
 
-  const { euiTheme } = useEuiTheme();
-  const buttonCss = css`
-    margin-top: -${euiTheme.size.base};
-    margin-bottom: 0;
-  `;
-
   return (
     <EuiResizableContainer
       className={className}
@@ -189,7 +178,7 @@ export const PanelsResizable = ({
             {topPanel}
           </EuiResizablePanel>
           <EuiResizableButton
-            css={[resizeWithPortalsHackButtonCss, buttonCss]}
+            css={resizeWithPortalsHackButtonCss}
             data-test-subj="unifiedHistogramResizableButton"
           />
           <EuiResizablePanel