diff --git a/maternal-web/app/settings/page.tsx b/maternal-web/app/settings/page.tsx index 561cb85..229040c 100644 --- a/maternal-web/app/settings/page.tsx +++ b/maternal-web/app/settings/page.tsx @@ -28,6 +28,7 @@ export default function SettingsPage() { notifications: true, emailUpdates: false, darkMode: false, + measurementUnit: 'metric' as 'metric' | 'imperial', }); const [isLoading, setIsLoading] = useState(false); const [error, setError] = useState(null); @@ -41,6 +42,7 @@ export default function SettingsPage() { notifications: user.preferences.notifications ?? true, emailUpdates: user.preferences.emailUpdates ?? false, darkMode: user.preferences.darkMode ?? false, + measurementUnit: (user.preferences.measurementUnit as 'metric' | 'imperial') || 'metric', }); setTimeFormat(user.preferences.timeFormat || '12h'); } @@ -169,7 +171,10 @@ export default function SettingsPage() { - + setSettings(prev => ({ ...prev, measurementUnit: value }))} + /> diff --git a/maternal-web/components/forms/UnitInput.tsx b/maternal-web/components/forms/UnitInput.tsx index deb72d2..d6bfac6 100644 --- a/maternal-web/components/forms/UnitInput.tsx +++ b/maternal-web/components/forms/UnitInput.tsx @@ -44,6 +44,9 @@ export function UnitInput({ type, value, onChange, ...textFieldProps }: UnitInpu const measurementSystem: MeasurementSystem = (user?.preferences?.measurementUnit as MeasurementSystem) || 'metric'; + // Debug logging + console.log('[UnitInput] Measurement system:', measurementSystem, 'User preferences:', user?.preferences); + // Get the display unit symbol const unitSymbol = getUnitSymbol(type, measurementSystem); diff --git a/maternal-web/components/settings/MeasurementUnitSelector.tsx b/maternal-web/components/settings/MeasurementUnitSelector.tsx index f9f82b5..3ece52e 100644 --- a/maternal-web/components/settings/MeasurementUnitSelector.tsx +++ b/maternal-web/components/settings/MeasurementUnitSelector.tsx @@ -17,6 +17,8 @@ interface MeasurementUnitSelectorProps { variant?: 'standard' | 'outlined' | 'filled'; fullWidth?: boolean; showDescription?: boolean; + value?: MeasurementSystem; + onChange?: (value: MeasurementSystem) => void; } /** @@ -26,19 +28,25 @@ export function MeasurementUnitSelector({ variant = 'outlined', fullWidth = true, showDescription = true, + value, + onChange, }: MeasurementUnitSelectorProps) { const { measurementSystem, setMeasurementSystem } = useLocale(); const { t } = useTranslation('settings'); - const [selectedSystem, setSelectedSystem] = useState(measurementSystem); - useEffect(() => { - setSelectedSystem(measurementSystem); - }, [measurementSystem]); + // Use controlled value if provided, otherwise use hook value + const selectedSystem = value !== undefined ? value : measurementSystem; const handleSystemChange = (event: SelectChangeEvent) => { const newSystem = event.target.value as MeasurementSystem; - setSelectedSystem(newSystem); - setMeasurementSystem(newSystem); + + // If onChange is provided (controlled), use it + if (onChange) { + onChange(newSystem); + } else { + // Otherwise use the hook (uncontrolled) + setMeasurementSystem(newSystem); + } }; return (