fixed the color detector to actually work. important to make functions do what you want them to do.

This commit is contained in:
Juicysteak117
2025-10-16 11:21:53 -07:00
parent 9520de3647
commit 607ef76589

View File

@@ -43,7 +43,6 @@ function copyURI(evt) {
// determines if the user has a set theme or a stored theme on load // determines if the user has a set theme or a stored theme on load
function detectColorScheme(){ function detectColorScheme(){
var theme="light"; //default to light var theme="light"; //default to light
localStorage.setItem('theme', 'light'); // not technically required
//local storage is used to override OS theme settings //local storage is used to override OS theme settings
if(localStorage.getItem("theme")){ if(localStorage.getItem("theme")){
if(localStorage.getItem("theme") == "dark"){ if(localStorage.getItem("theme") == "dark"){
@@ -63,6 +62,7 @@ function detectColorScheme(){
localStorage.setItem('theme', 'dark'); localStorage.setItem('theme', 'dark');
// now honestly not to criticize this code i am copying too much but like // now honestly not to criticize this code i am copying too much but like
// why not reuse the theme var here? surely it's more error prone this way? // why not reuse the theme var here? surely it's more error prone this way?
// is there some sort of perf diff in js this way? surely not? nerds, assemble
} }
} }
@@ -84,6 +84,7 @@ function initThemeToggle() {
} }
// the same function except it's for font toggle // the same function except it's for font toggle
// btw i think i'm a little funny for the var naming
function initFontToggle() { function initFontToggle() {
document.getElementById('font-toggle').addEventListener('click', () => { document.getElementById('font-toggle').addEventListener('click', () => {
document.documentElement.setAttribute( document.documentElement.setAttribute(