mirror of
https://github.com/MBombeck/HealthLog.git
synced 2026-11-08 12:32:43 +00:00
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.
This commit is contained in:
1 parent
b828d86749
commit
ae4bfc646e
1 file changed
+2
-2
@@ -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"
|
||||
>
|
||||
|
||||
Reference in new issue
Block a user