Skip to content

Commit f62ffa9

Browse files
committed
style(auth-ui): remove ThemeToggle and default to light theme
Remove the ThemeToggle component, its associated tests, and its remaining usage in the split layout. Update the root ThemeProvider to default to "light" rather than "system".
1 parent 651307e commit f62ffa9

5 files changed

Lines changed: 5 additions & 180 deletions

File tree

app/components/theme-toggle/__tests__/theme-toggle.test.tsx

Lines changed: 0 additions & 95 deletions
This file was deleted.

app/components/theme-toggle/theme-toggle.tsx

Lines changed: 0 additions & 78 deletions
This file was deleted.

app/layouts/split.layout.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import { ThemeToggle } from '@/components/theme-toggle/theme-toggle';
21
import type { BrandingTheme } from '@/modules/auth/types';
32
import { assetUrl } from '@/utils/asset-url';
43
import { Avatar, AvatarFallback, AvatarImage } from '@datum-cloud/datum-ui/avatar';
@@ -28,7 +27,6 @@ export default function SplitLayout({
2827
<Logo.Flat aria-label="Datum" className="h-6 w-auto" tone="brand" />
2928
)}
3029
</Link>
31-
<ThemeToggle />
3230
</div>
3331
<main className="flex w-full flex-1 items-center justify-center">
3432
<div className="w-full max-w-[400px]">{children}</div>

app/modules/i18n/locales/en.po

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ msgstr "Back"
123123
msgid "Back to sign in"
124124
msgstr "Back to sign in"
125125

126-
#: app/layouts/split.layout.tsx:38
126+
#: app/layouts/split.layout.tsx:36
127127
msgid "By continuing, you agree to Datum's <0>Terms of Service</0> and <1>Privacy Policy</1>, and to receive periodic emails with updates."
128128
msgstr "By continuing, you agree to Datum's <0>Terms of Service</0> and <1>Privacy Policy</1>, and to receive periodic emails with updates."
129129

@@ -437,7 +437,7 @@ msgstr "Phone sign-in isn't available — use your email or username."
437437
msgid "Please check your input and try again."
438438
msgstr "Please check your input and try again."
439439

440-
#: app/layouts/split.layout.tsx:106
440+
#: app/layouts/split.layout.tsx:104
441441
msgid "Prefer a demo instead? Just <0>reach out</0>."
442442
msgstr "Prefer a demo instead? Just <0>reach out</0>."
443443

@@ -594,7 +594,7 @@ msgstr "Something went wrong. Please try again."
594594
msgid "Start over"
595595
msgstr "Start over"
596596

597-
#: app/layouts/split.layout.tsx:118
597+
#: app/layouts/split.layout.tsx:116
598598
msgid "Thanks,"
599599
msgstr "Thanks,"
600600

@@ -680,7 +680,7 @@ msgstr "Use your security key to verify your identity."
680680
msgid "Username"
681681
msgstr "Username"
682682

683-
#: app/layouts/split.layout.tsx:96
683+
#: app/layouts/split.layout.tsx:94
684684
msgid "Using Datum requires setting up a billing account, but to help you explore without cost, we add <0>$50 USD</0> in credit on signup."
685685
msgstr "Using Datum requires setting up a billing account, but to help you explore without cost, we add <0>$50 USD</0> in credit on signup."
686686

app/root.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ export default function App() {
113113
);
114114

115115
return (
116-
<ThemeProvider attribute="class" defaultTheme="system" enableSystem disableTransitionOnChange>
116+
<ThemeProvider attribute="class" defaultTheme="light" enableSystem disableTransitionOnChange>
117117
<I18nProvider i18n={i18nInstance}>
118118
<ConformAdapter>
119119
<FathomAnalytics siteId={fathomSiteId} />

0 commit comments

Comments
 (0)