From dbe29156356273e337c07de2b74a4b528b9202cd Mon Sep 17 00:00:00 2001 From: Jack Kingsman Date: Thu, 19 Mar 2026 17:56:04 -0700 Subject: [PATCH] Use metric by default --- frontend/src/contexts/DistanceUnitContext.tsx | 2 +- frontend/src/test/distanceUnits.test.ts | 8 ++++---- frontend/src/test/settingsModal.test.tsx | 4 ++-- frontend/src/utils/distanceUnits.ts | 4 ++-- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/frontend/src/contexts/DistanceUnitContext.tsx b/frontend/src/contexts/DistanceUnitContext.tsx index e343b57..640b653 100644 --- a/frontend/src/contexts/DistanceUnitContext.tsx +++ b/frontend/src/contexts/DistanceUnitContext.tsx @@ -10,7 +10,7 @@ interface DistanceUnitContextValue { const noop = () => {}; const DistanceUnitContext = createContext({ - distanceUnit: 'imperial', + distanceUnit: 'metric', setDistanceUnit: noop, }); diff --git a/frontend/src/test/distanceUnits.test.ts b/frontend/src/test/distanceUnits.test.ts index 9aeffb8..16f8e26 100644 --- a/frontend/src/test/distanceUnits.test.ts +++ b/frontend/src/test/distanceUnits.test.ts @@ -11,8 +11,8 @@ describe('distanceUnits utilities', () => { localStorage.clear(); }); - it('defaults to imperial when unset', () => { - expect(getSavedDistanceUnit()).toBe('imperial'); + it('defaults to metric when unset', () => { + expect(getSavedDistanceUnit()).toBe('metric'); }); it('returns the stored unit when valid', () => { @@ -20,9 +20,9 @@ describe('distanceUnits utilities', () => { expect(getSavedDistanceUnit()).toBe('metric'); }); - it('falls back to imperial for invalid stored values', () => { + it('falls back to metric for invalid stored values', () => { localStorage.setItem(DISTANCE_UNIT_KEY, 'parsecs'); - expect(getSavedDistanceUnit()).toBe('imperial'); + expect(getSavedDistanceUnit()).toBe('metric'); }); it('stores the selected distance unit', () => { diff --git a/frontend/src/test/settingsModal.test.tsx b/frontend/src/test/settingsModal.test.tsx index 68b5a35..04f6e0a 100644 --- a/frontend/src/test/settingsModal.test.tsx +++ b/frontend/src/test/settingsModal.test.tsx @@ -536,12 +536,12 @@ describe('SettingsModal', () => { expect(localStorage.getItem(LAST_VIEWED_CONVERSATION_KEY)).toBeNull(); }); - it('defaults distance units to imperial and stores local changes', () => { + it('defaults distance units to metric and stores local changes', () => { renderModal(); openLocalSection(); const select = screen.getByLabelText('Distance Units'); - expect(select).toHaveValue('imperial'); + expect(select).toHaveValue('metric'); fireEvent.change(select, { target: { value: 'smoots' } }); diff --git a/frontend/src/utils/distanceUnits.ts b/frontend/src/utils/distanceUnits.ts index 4d52c03..30029eb 100644 --- a/frontend/src/utils/distanceUnits.ts +++ b/frontend/src/utils/distanceUnits.ts @@ -17,9 +17,9 @@ function isDistanceUnit(value: unknown): value is DistanceUnit { export function getSavedDistanceUnit(): DistanceUnit { try { const raw = localStorage.getItem(DISTANCE_UNIT_KEY); - return isDistanceUnit(raw) ? raw : 'imperial'; + return isDistanceUnit(raw) ? raw : 'metric'; } catch { - return 'imperial'; + return 'metric'; } }