fix: use resolved theme value (#1039)

This commit is contained in:
Ian Jones 2025-06-02 22:01:20 -04:00 committed by GitHub
parent 8fff778833
commit 7d8e71383f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 5 deletions

View file

@ -25,7 +25,7 @@ const PureSpreadsheetEditor = ({
status, status,
isCurrentVersion, isCurrentVersion,
}: SheetEditorProps) => { }: SheetEditorProps) => {
const { theme } = useTheme(); const { resolvedTheme } = useTheme();
const parseData = useMemo(() => { const parseData = useMemo(() => {
if (!content) return Array(MIN_ROWS).fill(Array(MIN_COLS).fill('')); if (!content) return Array(MIN_ROWS).fill(Array(MIN_COLS).fill(''));
@ -111,7 +111,7 @@ const PureSpreadsheetEditor = ({
return ( return (
<DataGrid <DataGrid
className={theme === 'dark' ? 'rdg-dark' : 'rdg-light'} className={resolvedTheme === 'dark' ? 'rdg-dark' : 'rdg-light'}
columns={columns} columns={columns}
rows={localRows} rows={localRows}
enableVirtualization enableVirtualization

View file

@ -26,7 +26,7 @@ import { guestRegex } from '@/lib/constants';
export function SidebarUserNav({ user }: { user: User }) { export function SidebarUserNav({ user }: { user: User }) {
const router = useRouter(); const router = useRouter();
const { data, status } = useSession(); const { data, status } = useSession();
const { setTheme, theme } = useTheme(); const { setTheme, resolvedTheme } = useTheme();
const isGuest = guestRegex.test(data?.user?.email ?? ''); const isGuest = guestRegex.test(data?.user?.email ?? '');
@ -74,9 +74,9 @@ export function SidebarUserNav({ user }: { user: User }) {
<DropdownMenuItem <DropdownMenuItem
data-testid="user-nav-item-theme" data-testid="user-nav-item-theme"
className="cursor-pointer" className="cursor-pointer"
onSelect={() => setTheme(theme === 'dark' ? 'light' : 'dark')} onSelect={() => setTheme(resolvedTheme === 'dark' ? 'light' : 'dark')}
> >
{`Toggle ${theme === 'light' ? 'dark' : 'light'} mode`} {`Toggle ${resolvedTheme === 'light' ? 'dark' : 'light'} mode`}
</DropdownMenuItem> </DropdownMenuItem>
<DropdownMenuSeparator /> <DropdownMenuSeparator />
<DropdownMenuItem asChild data-testid="user-nav-item-auth"> <DropdownMenuItem asChild data-testid="user-nav-item-auth">