fribyte-code / fribyte.no

Official website for friByte 🌐
https://fribyte.no
GNU General Public License v3.0
2 stars 8 forks source link

Dark mode #1

Closed Cengelsen closed 2 years ago

Cengelsen commented 2 years ago

We should implement a dark mode for "easier-on-the-eyes"-viewing.

timharek commented 2 years ago

Skal dette være basert på OSet og/eller skal vi bruke JavaScript sånn at brukeren kan overskrive og velge selv?

Cengelsen commented 2 years ago

Jeg hadde personlig foretrukket javascript som gir brukeren mulighet til å velge selv. :)

timharek commented 2 years ago

Jeg ser nå hvis vi skal ha JS for å kunne velge dark mode så mister vi automatisk dark mode hvis brukeren har deaktivert JS i nettleseren sin. Siden det ikke går an å overskrive prefers-color-scheme i JS, vi kan bare lytte til endringene :/

Hva tenker du om det @Cengelsen ?

Pr nå har #8 implementert riktige farger for dark mode, men ikke JS-logikken.

Cengelsen commented 2 years ago

I så tilfelle, så tenker jeg at det mest hensiktsmessige hadde vært å implementere automatisk mørk modus.

Men om det er mulig å implementere muligheten til å bytte mellom mørk og modus, med automatisk mørk modus som en hvilepute, så hadde det vært best. Da får de som ikke har blokkert js i nettleseren opp knappen til å velge, mens de som har blokkert js i nettleseren får sin foretrukne modus.

Om jeg leser det riktig, så er dette en implementasjon av det jeg tenker på.

timharek commented 2 years ago

Da får de som ikke har blokkert js i nettleseren opp knappen til å velge, mens de som har blokkert js i nettleseren får sin foretrukne modus.

Såvidt jeg vet er ikke det mulig å få begge deler, hvis man skal ha muligheten til å velge mellom lys- og mørkmodus med JS. Men en toggle er implementert i #8 nå, så den er klar for review! 😊 @Cengelsen

Cengelsen commented 2 years ago

Ok, nydelig! Har godkjent #8, så om du ikke har noe imot mine justeringer, så er det bare å slå grenene sammen @timharek :D