From 577adf7f13ca83af5109186345dc71b12ae40733 Mon Sep 17 00:00:00 2001
From: Richie <richie.phillips@proton.me>
Date: Mon, 30 Mar 2026 18:31:04 +1100
Subject: [PATCH] Retroactive review Phase 2.2: audit priority molecules, fix
 P0/P1

- AddOnOption: add aria-disabled when disabled (P1 a11y)
- ProviderCardCompact: add maxLines={1} on name to prevent overflow (P2 UX)
- ServiceOption: price uses labelLg variant instead of h6 + hardcoded fontWeight (P1 theming)

Audit scores: ServiceOption 13/20, AddOnOption 14/20, ProviderCardCompact 15/20

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 src/components/molecules/AddOnOption/AddOnOption.tsx          | 1 +
 .../molecules/ProviderCardCompact/ProviderCardCompact.tsx     | 2 +-
 src/components/molecules/ServiceOption/ServiceOption.tsx      | 4 ++--
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/components/molecules/AddOnOption/AddOnOption.tsx b/src/components/molecules/AddOnOption/AddOnOption.tsx
index 2b0940c..80bd88e 100644
--- a/src/components/molecules/AddOnOption/AddOnOption.tsx
+++ b/src/components/molecules/AddOnOption/AddOnOption.tsx
@@ -99,6 +99,7 @@ export const AddOnOption = React.forwardRef<HTMLDivElement, AddOnOptionProps>(
         selected={checked}
         padding="none"
         onClick={handleToggle}
+        aria-disabled={disabled || undefined}
         sx={[
           {
             p: 'var(--fa-card-padding-compact)',
diff --git a/src/components/molecules/ProviderCardCompact/ProviderCardCompact.tsx b/src/components/molecules/ProviderCardCompact/ProviderCardCompact.tsx
index 50980d7..ef605c7 100644
--- a/src/components/molecules/ProviderCardCompact/ProviderCardCompact.tsx
+++ b/src/components/molecules/ProviderCardCompact/ProviderCardCompact.tsx
@@ -95,7 +95,7 @@ export const ProviderCardCompact = React.forwardRef<HTMLDivElement, ProviderCard
             minWidth: 0,
           }}
         >
-          <Typography variant="h6" component="span">
+          <Typography variant="h6" component="span" maxLines={1}>
             {name}
           </Typography>
 
diff --git a/src/components/molecules/ServiceOption/ServiceOption.tsx b/src/components/molecules/ServiceOption/ServiceOption.tsx
index f184e84..9a2b5e1 100644
--- a/src/components/molecules/ServiceOption/ServiceOption.tsx
+++ b/src/components/molecules/ServiceOption/ServiceOption.tsx
@@ -116,10 +116,10 @@ export const ServiceOption = React.forwardRef<HTMLDivElement, ServiceOptionProps
           </Typography>
           {price != null && (
             <Typography
-              variant="h6"
+              variant="labelLg"
               component="span"
               color="primary"
-              sx={{ fontWeight: 600, whiteSpace: 'nowrap' }}
+              sx={{ whiteSpace: 'nowrap' }}
             >
               ${price.toLocaleString('en-AU')}
             </Typography>