diff --git a/apps/web/src/lib/api.ts b/apps/web/src/lib/api.ts index 2d4731b..8f68f89 100644 --- a/apps/web/src/lib/api.ts +++ b/apps/web/src/lib/api.ts @@ -7,8 +7,7 @@ * Fix: change the import to `useDebounce`. */ -// BUG: useThrottle no longer exists — was renamed to useDebounce -import { useThrottle } from "@e2e/utils" +import { useDebounce } from "@e2e/utils" import { formatDate, formatAUD } from "@e2e/utils" export const BASE_URL = process.env.API_URL ?? "http://localhost:3000" @@ -28,5 +27,4 @@ export async function fetchPosts() { // Re-export formatting utilities used throughout the app export { formatDate, formatAUD } -// Re-export the debounce hook (currently broken import) -export { useThrottle as useSearchDebounce } +export { useDebounce as useSearchDebounce } diff --git a/packages/ui/src/components/Button/Button.tsx b/packages/ui/src/components/Button/Button.tsx index af65c97..bf781b2 100644 --- a/packages/ui/src/components/Button/Button.tsx +++ b/packages/ui/src/components/Button/Button.tsx @@ -2,45 +2,28 @@ import React from "react" type Variant = "primary" | "secondary" | "danger" -type Props = { +type CustomProps = { children?: React.ReactNode - /** Icon-only button — renders without visible text. Requires aria-label for accessibility. */ icon?: React.ReactNode iconOnly?: boolean variant?: Variant - disabled?: boolean - onClick?: () => void - /** Accessible label — REQUIRED when iconOnly is true */ - "aria-label"?: string } -/** - * Button component. - * - * BUG: When `iconOnly` is true, the button renders without visible text. - * An `aria-label` is required for screen reader accessibility (WCAG 2.1 SC 4.1.2), - * but the component does not enforce or warn about its absence. - * - * The test in Button.test.tsx checks that an icon-only button has an accessible name. - * Fix: throw/warn in development when `iconOnly && !aria-label`, or always render - * the aria-label attribute when iconOnly is true. - */ +type Props = CustomProps & React.ButtonHTMLAttributes + export function Button({ children, icon, iconOnly = false, variant = "primary", - disabled = false, - onClick, "aria-label": ariaLabel, + ...rest }: Props) { return (