Balint66 / NeptunSkins

11 stars 4 forks source link

témák általánosítása #5

Open boapps opened 2 years ago

boapps commented 2 years ago

kulon alap temak:

boapps commented 2 years ago

mondjuk ha minden temanak kulon szineket definialunk, akkor lehet megse olyan fontos a kulon light/dark tema, max ikonok miatt lehet ertelme (bar azt is lehet szerintem tint-elni valamelyik valtozoval)

Balint66 commented 2 years ago

Az ikonokat fontawesome-al meg tudjuk oldani, mivel elvileg a Neptun használ 4.7.0-ás verziót (App_Themes/fonts/font-awesome.min.css). Ref: PTE ELTE OE MOME

Mivel ezek font-ok, talán a színezést is meg tudjuk oldani css-el, és akkor a jó kis png-s ikonoknak búcsút mondhatunk.

Balint66 commented 2 years ago

A színeket eddig is változókkal csináltuk, ha kell, simán tudunk definiálni még változókat, amiket bele tudunk építeni a css-be.

Az viszont lehet jó ötlet lenne, ha a header-eket is változókba raknánk, habár nem tudom, ez lehetséges-e, ennek utána kell járni.

Balint66 commented 2 years ago

A header-ekre kettő megoldást látok:

  1. Mivel url-be nem lehet használni a var()-t, így kénytelenek lennénk magát az url objektumot változóba menteni. Ugyanúgy tudunk abszolút meg relatív címeket megadni benne. Hiba lehetőséget nem látok benne, csak bennem annyi kérdés merül fel, hogy jól kezeli-e majd a relatív elérési utat.
  2. Jó öreg javascript

Az első megoldást preferálnám, szerintem igen elegáns lenne.

Balint66 commented 2 years ago

Az ikonokat fontawesome-al meg tudjuk oldani, mivel elvileg a Neptun használ 4.7.0-ás verziót (App_Themes/fonts/font-awesome.min.css). Ref: PTE ELTE OE MOME

Mivel ezek font-ok, talán a színezést is meg tudjuk oldani css-el, és akkor a jó kis png-s ikonoknak búcsút mondhatunk.

Mivel nem igazán láttam, hogy bárhol nagyon használná (talán csak benne felejtették egy tesztelés során?) lehetne akár, hogy egy frisebbet használunk, csak ez lehet a betöltési időhöz még többet hozzá tenne.