Skip to content

Commit fa0a597

Browse files
authored
fix: add build status favicons based on system theme (#10089)
1 parent f270d9d commit fa0a597

17 files changed

+26
-2
lines changed

site/src/pages/WorkspacePage/WorkspaceReadyPage.tsx

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,14 @@ export const WorkspaceReadyPage = ({
9898
...templateVersion(workspace.template_active_version_id),
9999
enabled: workspace.outdated,
100100
});
101+
const systemTheme = useState(() => {
102+
if (typeof window.matchMedia === "undefined") {
103+
// Default to dark mode!
104+
return "dark";
105+
}
106+
const isDark = window.matchMedia("(prefers-color-scheme: dark)");
107+
return isDark ? "dark" : "light";
108+
});
101109

102110
const buildLogs = useWorkspaceBuildLogs(workspace.latest_build.id);
103111
const shouldDisplayBuildLogs =
@@ -124,12 +132,12 @@ export const WorkspaceReadyPage = ({
124132
<link
125133
rel="alternate icon"
126134
type="image/png"
127-
href={`/favicons/${favicon}.png`}
135+
href={`/favicons/${favicon}-${systemTheme}.png`}
128136
/>
129137
<link
130138
rel="icon"
131139
type="image/svg+xml"
132-
href={`/favicons/${favicon}.svg`}
140+
href={`/favicons/${favicon}-${systemTheme}.svg`}
133141
/>
134142
</Helmet>
135143

2.49 KB
Loading
Lines changed: 4 additions & 0 deletions
Loading
2.46 KB
Loading
Lines changed: 4 additions & 0 deletions
Loading
2.42 KB
Loading
Lines changed: 4 additions & 0 deletions
Loading
2.4 KB
Loading
Lines changed: 4 additions & 0 deletions
Loading

0 commit comments

Comments
 (0)