Arsharth/hydrogen-web
1
0
Fork 0
forked from mystiq/hydrogen-web
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #751 from vector-im/fix-theming-watch

Browse source 
Fix: don't crash on platforms that don't have a preferred color scheme
This commit is contained in:
Bruno Windels 2022-06-14 16:00:13 +02:00 committed by GitHub
commit e643ffb334
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/domain/session/settings/SettingsViewModel.js
View file

@ -187,9 +187,5 @@ export class SettingsViewModel extends ViewModel {
// emit so that radio-buttons become displayed/hidden // emit so that radio-buttons become displayed/hidden
this.emitChange("themeOption"); this.emitChange("themeOption");
} }
get preferredColorScheme() {
return this.platform.themeLoader.preferredColorScheme;
}
} }

3
src/platform/web/ThemeLoader.ts
View file

@ -196,13 +196,12 @@ export class ThemeLoader {
} }
} }
get preferredColorScheme(): ColorSchemePreference { get preferredColorScheme(): ColorSchemePreference | undefined {
if (window.matchMedia("(prefers-color-scheme: dark)").matches) { if (window.matchMedia("(prefers-color-scheme: dark)").matches) {
return ColorSchemePreference.Dark; return ColorSchemePreference.Dark;
} }
else if (window.matchMedia("(prefers-color-scheme: light)").matches) { else if (window.matchMedia("(prefers-color-scheme: light)").matches) {
return ColorSchemePreference.Light; return ColorSchemePreference.Light;
} }
throw new Error("Cannot find preferred colorscheme!");
} }
} }