- {/* Raised, not panel: this section is a band, which is itself
- bg-panel — a panel-on-panel callout has only a 1.36:1 hairline
- holding it together. */}
Your downloads and your shared directories are yours: they stay exactly where
diff --git a/website/src/theme/theme-context.ts b/website/src/theme/theme-context.ts
index 9b35301..5b29849 100644
--- a/website/src/theme/theme-context.ts
+++ b/website/src/theme/theme-context.ts
@@ -8,18 +8,11 @@ export const isTheme = (value: unknown): value is Theme =>
typeof value === 'string' && (THEMES as readonly string[]).includes(value)
export interface ThemeContextValue {
- /** What the user picked — may be 'system'. */
theme: Theme
- /** What is actually painted right now — never 'system'. */
resolvedTheme: ResolvedTheme
setTheme: (theme: Theme) => void
}
-/*
- * Split from the provider so this module exports only values, which keeps the
- * provider file component-only for react-refresh.
- */
export const ThemeContext = createContext(null)
-/** Matches the key the design's own inline boot script reads. */
export const THEME_STORAGE_KEY = 'ssrs-theme'
diff --git a/website/src/theme/theme-provider.tsx b/website/src/theme/theme-provider.tsx
index dca9a00..01f5bd1 100644
--- a/website/src/theme/theme-provider.tsx
+++ b/website/src/theme/theme-provider.tsx
@@ -25,8 +25,6 @@ export const ThemeProvider = ({ children }: ThemeProviderProps) => {
useEffect(() => {
const root = document.documentElement
root.dataset['theme'] = resolvedTheme
- // Paints native widgets (scrollbars, form controls) to match, which the
- // attribute alone does not do.
root.style.colorScheme = resolvedTheme
}, [resolvedTheme])
diff --git a/website/src/theme/theme-store.ts b/website/src/theme/theme-store.ts
index 42eccb1..35a9fb8 100644
--- a/website/src/theme/theme-store.ts
+++ b/website/src/theme/theme-store.ts
@@ -2,21 +2,8 @@ import { isTheme, THEME_STORAGE_KEY, type ResolvedTheme, type Theme } from './th
const DARK_QUERY = '(prefers-color-scheme: dark)'
-/*
- * The theme lives outside React: the boot script in index.html resolves and
- * paints it before the bundle loads, and the OS can change it at any time.
- * That makes it an external store, which is what useSyncExternalStore is for
- * — and it is the piece that gives correct hydration for free.
- *
- * The server snapshots below must match what the prerendered HTML ships
- * (``). React renders the server snapshot for the
- * hydration pass, then immediately re-renders with the real one, so a visitor
- * whose actual theme is light never sees a mismatch, only a corrected render.
- */
-
const listeners = new Set<() => void>()
-/** Notifies subscribers after a local write; OS changes come via matchMedia. */
const emit = () => {
for (const listener of listeners) listener()
}
@@ -25,7 +12,6 @@ export const subscribe = (onChange: () => void): (() => void) => {
listeners.add(onChange)
const query = window.matchMedia(DARK_QUERY)
query.addEventListener('change', onChange)
- // Another tab changing the preference should move this one too.
window.addEventListener('storage', onChange)
return () => {
listeners.delete(onChange)
@@ -39,7 +25,6 @@ export const readTheme = (): Theme => {
const stored = localStorage.getItem(THEME_STORAGE_KEY)
return isTheme(stored) ? stored : 'system'
} catch {
- // Safari in private mode throws on localStorage access.
return 'system'
}
}
@@ -50,15 +35,16 @@ export const readResolvedTheme = (): ResolvedTheme => {
return window.matchMedia(DARK_QUERY).matches ? 'dark' : 'light'
}
-/** Both must agree with the `data-theme` baked into index.html. */
export const serverTheme = (): Theme => 'system'
export const serverResolvedTheme = (): ResolvedTheme => 'dark'
-export const writeTheme = (next: Theme): void => {
+export const writeTheme = (next: Theme): boolean => {
+ let persisted = true
try {
localStorage.setItem(THEME_STORAGE_KEY, next)
} catch {
- // Not persisting is survivable; the theme still applies this session.
+ persisted = false
}
emit()
+ return persisted
}
diff --git a/website/src/vite-env.d.ts b/website/src/vite-env.d.ts
index daac02e..54eaa07 100644
--- a/website/src/vite-env.d.ts
+++ b/website/src/vite-env.d.ts
@@ -1,4 +1,3 @@
///
-/** Injected by `define` in vite.config.ts from the workspace Cargo.toml. */
declare const __APP_VERSION__: string
diff --git a/website/tsconfig.app.json b/website/tsconfig.app.json
index f55c377..a8c278b 100644
--- a/website/tsconfig.app.json
+++ b/website/tsconfig.app.json
@@ -15,7 +15,6 @@
"resolveJsonModule": true,
"noEmit": true,
- /* Strictest. Every flag below is on deliberately — see README. */
"strict": true,
"noUncheckedIndexedAccess": true,
"exactOptionalPropertyTypes": true,
@@ -31,7 +30,6 @@
"skipLibCheck": true,
- /* No baseUrl — deprecated in TS 6. `paths` resolves relative to this file. */
"paths": {
"@/*": ["./src/*"]
}
diff --git a/website/tsconfig.node.json b/website/tsconfig.node.json
index ed0b029..e6d6a26 100644
--- a/website/tsconfig.node.json
+++ b/website/tsconfig.node.json
@@ -10,7 +10,6 @@
"isolatedModules": true,
"verbatimModuleSyntax": true,
"erasableSyntaxOnly": true,
- /* scripts/ is run straight from source by Bun, which needs the extension. */
"allowImportingTsExtensions": true,
"noEmit": true,
diff --git a/website/types/eslint-plugin-jsx-a11y.d.ts b/website/types/eslint-plugin-jsx-a11y.d.ts
index 31011b0..a7365f4 100644
--- a/website/types/eslint-plugin-jsx-a11y.d.ts
+++ b/website/types/eslint-plugin-jsx-a11y.d.ts
@@ -1,9 +1,3 @@
-/*
- * eslint-plugin-jsx-a11y ships no types, and TS's inference from its JS picks
- * up the legacy eslintrc `configs` shape (plugins as string[]) instead of the
- * flat-config one. Declaring the flat export is enough to type the one thing
- * eslint.config.ts touches.
- */
declare module 'eslint-plugin-jsx-a11y' {
import type { Linter } from 'eslint'
diff --git a/website/vite.config.ts b/website/vite.config.ts
index 95be054..ebbc59b 100644
--- a/website/vite.config.ts
+++ b/website/vite.config.ts
@@ -5,11 +5,6 @@ import tailwindcss from '@tailwindcss/vite'
import react from '@vitejs/plugin-react'
import { defineConfig } from 'vite'
-/*
- * Read the version out of the workspace manifest rather than keeping a copy in
- * TypeScript. release-plz bumps Cargo.toml; a hand-written constant here would
- * silently advertise the previous release forever.
- */
const cargoToml = readFileSync(
fileURLToPath(new URL('../Cargo.toml', import.meta.url)),
'utf8',
@@ -19,10 +14,6 @@ if (versionMatch?.[1] === undefined)
throw new Error('could not read version from ../Cargo.toml')
const VERSION = `v${versionMatch[1]}`
-// Project sites are served from https://.github.io//, so every
-// asset URL needs that prefix baked in at build time.
-// hardcoded. Point a custom domain at the site and this becomes '/'
-// — one edit here plus the matching `basename` in src/main.tsx.
const BASE = '/soulseek-rs/'
export default defineConfig({
@@ -37,8 +28,6 @@ export default defineConfig({
},
},
build: {
- // Small site; a warning at the default 500 kB would be noise. A jump past
- // this means something heavy got imported by accident.
chunkSizeWarningLimit: 600,
},
})