From d16dbc5ac94881491b9b6454b431aa2cb43a9654 Mon Sep 17 00:00:00 2001 From: Daniel Alvo <48444638+DanMPA@users.noreply.github.com> Date: Fri, 4 Apr 2025 15:19:08 +0100 Subject: [PATCH] ColorPicker: Fixed height when switching tabs (#103304) * ColorPicker: fixed height when switching tabs Fixed the component's height by setting a consistent height and adding margin. Also added a test to ensure the component heights remain nearly identical when switching tabs. Fix #99675 * ColorPickerPopover: Fixed height for popover content and allowed overflow. #99675 * ColorPickerPopover : Fix formatting suggested by linter. * ColorPickerPopover: Remove redundant height test when switching tabs * ColorPickerPopover: Remove overflowY style from popover container * ColorPicker: Remove unnecessary marginTop from SpectrumPalette styles --- .../grafana-ui/src/components/ColorPicker/ColorPickerPopover.tsx | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/grafana-ui/src/components/ColorPicker/ColorPickerPopover.tsx b/packages/grafana-ui/src/components/ColorPicker/ColorPickerPopover.tsx index 1f51cf02cf1..7cf557f3c26 100644 --- a/packages/grafana-ui/src/components/ColorPicker/ColorPickerPopover.tsx +++ b/packages/grafana-ui/src/components/ColorPicker/ColorPickerPopover.tsx @@ -151,6 +151,7 @@ const getStyles = stylesFactory((theme: GrafanaTheme2) => { width: '246px', fontSize: theme.typography.bodySmall.fontSize, minHeight: '184px', + height: '290px', padding: theme.spacing(1), display: 'flex', flexDirection: 'column',