Skip to content

Commit d0a8424

Browse files
zxt-tzxzxt-tzx
authored
feat: remove dark blue theme (coder#14890)
1 parent 0589267 commit d0a8424

File tree

12 files changed

+4
-402
lines changed

12 files changed

+4
-402
lines changed

‎coderd/database/migrations/000260_remove_dark_blue_theme.down.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
-- Nothing to restore

‎coderd/database/migrations/000260_remove_dark_blue_theme.up.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
UPDATE users SET theme_preference = '' WHERE theme_preference = 'darkBlue';

‎site/src/pages/UserSettingsPage/AppearancePage/AppearanceForm.tsx

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -42,19 +42,12 @@ export const AppearanceForm: FC<AppearanceFormProps> = ({
4242
themes={[themes.dark, themes.light]}
4343
onSelect={() => onChangeTheme("auto")}
4444
/>
45-
4645
<ThemePreviewButton
4746
displayName="Dark"
4847
active={currentTheme === "dark"}
4948
theme={themes.dark}
5049
onSelect={() => onChangeTheme("dark")}
5150
/>
52-
<ThemePreviewButton
53-
displayName="Dark blue"
54-
active={currentTheme === "darkBlue"}
55-
theme={themes.darkBlue}
56-
onSelect={() => onChangeTheme("darkBlue")}
57-
/>
5851
<ThemePreviewButton
5952
displayName="Light"
6053
active={currentTheme === "light"}

‎site/src/pages/UserSettingsPage/AppearancePage/AppearancePage.test.tsx

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -21,24 +21,6 @@ describe("appearance page", () => {
2121
expect(API.updateAppearanceSettings).toBeCalledTimes(0);
2222
});
2323

24-
it("changes theme to dark blue", async () => {
25-
renderWithAuth(<AppearancePage />);
26-
27-
jest.spyOn(API, "updateAppearanceSettings").mockResolvedValueOnce({
28-
...MockUser,
29-
theme_preference: "darkBlue",
30-
});
31-
32-
const darkBlue = await screen.findByText("Dark blue");
33-
await userEvent.click(darkBlue);
34-
35-
// Check if the API was called correctly
36-
expect(API.updateAppearanceSettings).toBeCalledTimes(1);
37-
expect(API.updateAppearanceSettings).toHaveBeenCalledWith("me", {
38-
theme_preference: "darkBlue",
39-
});
40-
});
41-
4224
it("changes theme to light", async () => {
4325
renderWithAuth(<AppearancePage />);
4426

‎site/src/pages/UsersPage/storybookData/users.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -343,7 +343,7 @@ export const MockUsers: User[] = [
343343
last_seen_at: "2024-09-05T12:27:22.098297Z",
344344
status: "active",
345345
login_type: "oidc",
346-
theme_preference: "darkBlue",
346+
theme_preference: "dark",
347347
organization_ids: ["703f72a1-76f6-4f89-9de6-8a3989693fe5"],
348348
roles: [
349349
{
@@ -423,7 +423,7 @@ export const MockUsers: User[] = [
423423
last_seen_at: "2024-09-05T07:24:39.324649Z",
424424
status: "active",
425425
login_type: "oidc",
426-
theme_preference: "darkBlue",
426+
theme_preference: "dark",
427427
organization_ids: ["703f72a1-76f6-4f89-9de6-8a3989693fe5"],
428428
roles: [
429429
{

‎site/src/theme/darkBlue/branding.ts

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

‎site/src/theme/darkBlue/experimental.ts

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

‎site/src/theme/darkBlue/index.ts

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

‎site/src/theme/darkBlue/monaco.ts

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

‎site/src/theme/darkBlue/mui.ts

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

0 commit comments

Comments
 (0)