From ae4bfc646eadc9702b7bd12f35d3beba58651afb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= <mbombeck@gmail.com>
Date: Thu, 21 May 2026 22:05:36 +0200
Subject: [PATCH] a11y(charts): use responsive sizing on chart-overlay-controls
 trigger

The dropdown trigger combined `size="sm"` (32 px default) with a
forced `min-h-11 min-w-11` stretch. That was the v1.4.27 pattern;
v1.4.33 settled on the cleaner `size="default" + min-h-11 sm:min-h-9`
shape (so mobile hits 44 px and desktop drops to 36 px without the
stretched-button look the maintainer earlier called klobig). Adopt
the consistent pattern here.
---
 src/components/charts/chart-overlay-controls.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/charts/chart-overlay-controls.tsx b/src/components/charts/chart-overlay-controls.tsx
index 66d633b95..b05542878 100644
--- a/src/components/charts/chart-overlay-controls.tsx
+++ b/src/components/charts/chart-overlay-controls.tsx
@@ -98,8 +98,8 @@ export function ChartOverlayControls({
         <Button
           type="button"
           variant="ghost"
-          size="sm"
-          className={`text-muted-foreground hover:text-foreground min-h-11 min-w-11 px-0 ${triggerClassName ?? ""}`.trim()}
+          size="default"
+          className={`text-muted-foreground hover:text-foreground min-h-11 min-w-11 px-0 sm:min-h-9 sm:min-w-9 ${triggerClassName ?? ""}`.trim()}
           aria-label={t("chart.overlay.controls.tooltip.openSettings")}
           data-slot="chart-overlay-controls-trigger"
         >