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>
This commit is contained in:
@@ -99,6 +99,7 @@ export const AddOnOption = React.forwardRef<HTMLDivElement, AddOnOptionProps>(
|
|||||||
selected={checked}
|
selected={checked}
|
||||||
padding="none"
|
padding="none"
|
||||||
onClick={handleToggle}
|
onClick={handleToggle}
|
||||||
|
aria-disabled={disabled || undefined}
|
||||||
sx={[
|
sx={[
|
||||||
{
|
{
|
||||||
p: 'var(--fa-card-padding-compact)',
|
p: 'var(--fa-card-padding-compact)',
|
||||||
|
|||||||
@@ -95,7 +95,7 @@ export const ProviderCardCompact = React.forwardRef<HTMLDivElement, ProviderCard
|
|||||||
minWidth: 0,
|
minWidth: 0,
|
||||||
}}
|
}}
|
||||||
>
|
>
|
||||||
<Typography variant="h6" component="span">
|
<Typography variant="h6" component="span" maxLines={1}>
|
||||||
{name}
|
{name}
|
||||||
</Typography>
|
</Typography>
|
||||||
|
|
||||||
|
|||||||
@@ -116,10 +116,10 @@ export const ServiceOption = React.forwardRef<HTMLDivElement, ServiceOptionProps
|
|||||||
</Typography>
|
</Typography>
|
||||||
{price != null && (
|
{price != null && (
|
||||||
<Typography
|
<Typography
|
||||||
variant="h6"
|
variant="labelLg"
|
||||||
component="span"
|
component="span"
|
||||||
color="primary"
|
color="primary"
|
||||||
sx={{ fontWeight: 600, whiteSpace: 'nowrap' }}
|
sx={{ whiteSpace: 'nowrap' }}
|
||||||
>
|
>
|
||||||
${price.toLocaleString('en-AU')}
|
${price.toLocaleString('en-AU')}
|
||||||
</Typography>
|
</Typography>
|
||||||
|
|||||||
Reference in New Issue
Block a user