Skip to content

Commit a565745

Browse files
Bikram GoleBikram Gole
authored andcommitted
Make saved theme override stale URL theme across pages
1 parent 7be1c27 commit a565745

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

about.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
const params = new URLSearchParams(window.location.search);
2020
const urlTheme = params.get("theme");
2121
const savedTheme = window.localStorage.getItem(themeKey);
22-
const selected = validThemes.has(urlTheme) ? urlTheme : (validThemes.has(savedTheme) ? savedTheme : null);
22+
const selected = validThemes.has(savedTheme) ? savedTheme : (validThemes.has(urlTheme) ? urlTheme : null);
2323
if (selected) {
2424
document.body.dataset.theme = selected;
2525
if (selected !== savedTheme) {

contact.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
const params = new URLSearchParams(window.location.search);
2020
const urlTheme = params.get("theme");
2121
const savedTheme = window.localStorage.getItem(themeKey);
22-
const selected = validThemes.has(urlTheme) ? urlTheme : (validThemes.has(savedTheme) ? savedTheme : null);
22+
const selected = validThemes.has(savedTheme) ? savedTheme : (validThemes.has(urlTheme) ? urlTheme : null);
2323
if (selected) {
2424
document.body.dataset.theme = selected;
2525
if (selected !== savedTheme) {

index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
const params = new URLSearchParams(window.location.search);
2020
const urlTheme = params.get("theme");
2121
const savedTheme = window.localStorage.getItem(themeKey);
22-
const selected = validThemes.has(urlTheme) ? urlTheme : (validThemes.has(savedTheme) ? savedTheme : null);
22+
const selected = validThemes.has(savedTheme) ? savedTheme : (validThemes.has(urlTheme) ? urlTheme : null);
2323
if (selected) {
2424
document.body.dataset.theme = selected;
2525
if (selected !== savedTheme) {

script.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -625,8 +625,8 @@ function initThemeSwitcher() {
625625
try {
626626
urlTheme = getThemeFromUrl();
627627
const storedTheme = window.localStorage.getItem(THEME_STORAGE_KEY);
628-
savedTheme = urlTheme || storedTheme || "neo";
629-
if (urlTheme && storedTheme !== urlTheme) {
628+
savedTheme = storedTheme || urlTheme || "neo";
629+
if (!storedTheme && urlTheme) {
630630
window.localStorage.setItem(THEME_STORAGE_KEY, urlTheme);
631631
}
632632
} catch (error) {
@@ -637,7 +637,7 @@ function initThemeSwitcher() {
637637
window.addEventListener("pageshow", () => {
638638
let latestTheme = "neo";
639639
try {
640-
latestTheme = getThemeFromUrl() || window.localStorage.getItem(THEME_STORAGE_KEY) || "neo";
640+
latestTheme = window.localStorage.getItem(THEME_STORAGE_KEY) || getThemeFromUrl() || "neo";
641641
} catch (error) {
642642
latestTheme = "neo";
643643
}

0 commit comments

Comments
 (0)