From b5d46b326d90f96e0605c20f39bf2ee36f28cc9d Mon Sep 17 00:00:00 2001 From: BigSimmo <87357024+BigSimmo@users.noreply.github.com> Date: Thu, 27 Aug 2026 18:13:30 +0800 Subject: [PATCH 01/16] fix(ui): raise remaining interactive tap floors Primary actions on documents, compare, tools, calculators, forms, and chips still used sub-48px heights. Raise those controls to the tap floor so phone targets stay reachable without shrinking CTAs. Co-authored-by: Cursor --- src/components/calculators/calculator-ui.tsx | 6 +-- .../calculators/clinical-console.tsx | 4 +- .../DocumentManagerPanel.tsx | 14 +++---- .../clinical-dashboard/document-admin.tsx | 37 ++++++++----------- .../clinical-dashboard/document-ui.tsx | 2 +- .../clinical-dashboard/settings-dialog.tsx | 30 ++++++++------- .../compare/compare-catalog-picker.tsx | 2 +- .../compare/compare-empty-state.tsx | 2 +- src/components/forms/form-detail-page.tsx | 2 +- .../forms/forms-search-results-page.tsx | 8 ++-- .../tools/tools-search-results-page.tsx | 2 +- src/components/ui/chip.tsx | 14 +++---- 12 files changed, 60 insertions(+), 63 deletions(-) diff --git a/src/components/calculators/calculator-ui.tsx b/src/components/calculators/calculator-ui.tsx index 912cce3a59..a0be43d863 100644 --- a/src/components/calculators/calculator-ui.tsx +++ b/src/components/calculators/calculator-ui.tsx @@ -456,7 +456,7 @@ export function ResetButton({ onReset, disabled }: { onReset: () => void; disabl onClick={onReset} disabled={disabled} className={cn( - "inline-flex min-h-10 items-center gap-2 rounded-lg border border-[color:var(--border)] bg-[color:var(--surface)] px-3 text-sm-minus font-bold text-[color:var(--text-muted)] transition hover:border-[color:var(--border-strong)] hover:text-[color:var(--text)] disabled:pointer-events-none disabled:opacity-40", + "inline-flex min-h-tap items-center gap-2 rounded-lg border border-[color:var(--border)] bg-[color:var(--surface)] px-3 text-sm-minus font-bold text-[color:var(--text-muted)] transition hover:border-[color:var(--border-strong)] hover:text-[color:var(--text)] disabled:pointer-events-none disabled:opacity-40", focusRing, )} > @@ -512,7 +512,7 @@ export function CopyResultButton({ onClick={copy} disabled={!state.started} className={cn( - "inline-flex min-h-10 items-center gap-1.5 rounded-lg border border-[color:var(--border)] bg-[color:var(--surface)] px-2.5 text-2xs font-bold text-[color:var(--text-muted)] transition hover:border-[color:var(--border-strong)] hover:text-[color:var(--text)] disabled:pointer-events-none disabled:opacity-40", + "inline-flex min-h-tap items-center gap-1.5 rounded-lg border border-[color:var(--border)] bg-[color:var(--surface)] px-2.5 text-2xs font-bold text-[color:var(--text-muted)] transition hover:border-[color:var(--border-strong)] hover:text-[color:var(--text)] disabled:pointer-events-none disabled:opacity-40", focusRing, className, )} @@ -642,7 +642,7 @@ export function CalculatorItems({ type="button" onClick={() => onAnswersChange(seedCheckboxDefaults(calc, answers))} className={cn( - "mt-1 inline-flex min-h-10 items-center justify-center gap-2 rounded-lg border border-dashed border-[color:var(--border-strong)] bg-[color:var(--surface-subtle)] px-3 text-sm-minus font-bold text-[color:var(--text-muted)] transition hover:border-[color:var(--clinical-accent-border)] hover:text-[color:var(--text)]", + "mt-1 inline-flex min-h-tap items-center justify-center gap-2 rounded-lg border border-dashed border-[color:var(--border-strong)] bg-[color:var(--surface-subtle)] px-3 text-sm-minus font-bold text-[color:var(--text-muted)] transition hover:border-[color:var(--clinical-accent-border)] hover:text-[color:var(--text)]", focusRing, )} > diff --git a/src/components/calculators/clinical-console.tsx b/src/components/calculators/clinical-console.tsx index 57a84de8b1..ea628dffc8 100644 --- a/src/components/calculators/clinical-console.tsx +++ b/src/components/calculators/clinical-console.tsx @@ -129,7 +129,7 @@ export function CalculatorsClinicalConsoleMockup() { aria-pressed={active} onClick={() => setActiveId(entry.id)} className={cn( - "inline-flex min-h-10 shrink-0 items-center gap-2 rounded-md border px-3 text-sm-minus font-bold", + "inline-flex min-h-tap shrink-0 items-center gap-2 rounded-md border px-3 text-sm-minus font-bold", active ? "border-[color:var(--clinical-accent-border)] bg-[color:var(--clinical-accent-soft)] text-[color:var(--clinical-accent)]" : "border-[color:var(--border)] bg-[color:var(--surface)] text-[color:var(--text-muted)]", @@ -231,7 +231,7 @@ export function CalculatorsClinicalConsoleMockup() { onClick={() => setAnswers({})} disabled={!derived.started} className={cn( - "inline-flex min-h-9 items-center rounded-md border border-[color:var(--border)] bg-[color:var(--surface)] px-2.5 text-2xs font-bold text-[color:var(--text-muted)] hover:text-[color:var(--text)] disabled:pointer-events-none disabled:opacity-40", + "inline-flex min-h-tap items-center rounded-md border border-[color:var(--border)] bg-[color:var(--surface)] px-2.5 text-2xs font-bold text-[color:var(--text-muted)] hover:text-[color:var(--text)] disabled:pointer-events-none disabled:opacity-40", focusRing, )} > diff --git a/src/components/clinical-dashboard/DocumentManagerPanel.tsx b/src/components/clinical-dashboard/DocumentManagerPanel.tsx index 4191e10196..31e0c7a94c 100644 --- a/src/components/clinical-dashboard/DocumentManagerPanel.tsx +++ b/src/components/clinical-dashboard/DocumentManagerPanel.tsx @@ -201,7 +201,7 @@ export function IndexingMonitor({ type="button" onClick={() => onRetry(job.id)} disabled={busy} - className={cn(floatingControl, "min-h-9 px-3 text-xs")} + className={cn(floatingControl, "px-3 text-xs")} > {busy ? (