From 920520a2f695391f871b3d293d3e5a771ff617df Mon Sep 17 00:00:00 2001 From: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Date: Wed, 24 Sep 2025 13:58:32 +0000 Subject: [PATCH] Fix BUS-1891: Apply consistent 400px max height to all metric types - Remove special case for tables in cardClass logic - Apply max-h-[400px] universally instead of max-h-[600px] - Tables will now naturally size to content up to 400px limit - Maintains virtualization performance within height constraint Co-Authored-By: nate@buster.so --- .../features/metrics/MetricChartCard/MetricChartCard.tsx | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/apps/web/src/components/features/metrics/MetricChartCard/MetricChartCard.tsx b/apps/web/src/components/features/metrics/MetricChartCard/MetricChartCard.tsx index 6967bbe5f..398ae53cd 100644 --- a/apps/web/src/components/features/metrics/MetricChartCard/MetricChartCard.tsx +++ b/apps/web/src/components/features/metrics/MetricChartCard/MetricChartCard.tsx @@ -152,10 +152,9 @@ const MetricViewChartCardContainer = React.forwardRef< MetricViewChartCardContainerProps >(({ children, loadingData, hasData, errorData, isTable, className }, ref) => { const cardClass = React.useMemo(() => { - if (loadingData || errorData || !hasData) return 'h-full max-h-[600px]'; - if (isTable) return ''; - return 'h-full max-h-[600px]'; - }, [isTable, loadingData, hasData, errorData]); + if (loadingData || errorData || !hasData) return 'h-full max-h-[400px]'; + return 'h-full max-h-[400px]'; + }, [loadingData, hasData, errorData]); return (