search

Project-insert-name / root-website

Frontend application for rootlinjeforening.no
https://rootlinjeforening.no/
4 stars 0 forks source link

Redesign #162

Closed simsine closed 3 months ago

simsine commented 6 months ago

Beskriv endringene

Dette er en ganske stor redesign og forbedringer avde viktigste delene av nettsiden.

Relaterte issues

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
root-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 14, 2024 10:19am
simsine commented 3 months ago

@emberal Bra implementasjon av mørkt tema, i like. Vi kan vurdere å fjerne ubrukte logo assets fra public mappen før produksjonsetting. Bortsett fra det ser det bra ut for min del :+1:

olav223 commented 3 months ago

Er det meningen logoen skal se slik ut? image image

olav223 commented 3 months ago

Ellers ser det meste bra ut. Er det mulig å deploye dette i et testmiljø @simsine?

simsine commented 3 months ago

Ellers ser det meste bra ut. Er det mulig å deploye dette i et testmiljø @simsine?

IMO gir det ikke helt mening å deploye et standalone testmiljø for dette prosjektet. Det eneste vi tester er egentlig bare en frontend og det er ikke så mye ting å teste annet enn utseende og oppførsel av den. Å ha et standalone testmiljø vil enten inkludere ekstra manuelt arbeid via å manuelt bygge test images som kan deployes til serveren for å sees på av andre uten å selv kjøre applikasjonen. Ellers ville det inkludert et ekstra repo management steg ved å ha en egen test branch som man kan merge med for å teste som så blir merget til main, men dette vil også skape en vanskeligere prosess for utvikling som er vanskelig å opprettholde. Dette ville jo vært en mulighet, men for vår skala tror jeg det vil være unødvendig komplisert.

simsine commented 3 months ago

De som ville hatt bruk for et slikt test miljø er jo nøyaktig de personene som har sitt eget utviklingsmiljø å teste applikasjonen i selv

emberal commented 3 months ago

Er det meningen logoen skal se slik ut? image image

Problemet er at om man viser hele logoen, så blir den veldig liten, så det er zoomet litt for å få fram midten av logoen. Enig at det ser litt rart ut da, men usikker på hva som blir best måte å løse det på.

olav223 commented 3 months ago

Er det meningen logoen skal se slik ut? image image

Problemet er at om man viser hele logoen, så blir den veldig liten, så det er zoomet litt for å få fram midten av logoen. Enig at det ser litt rart ut da, men usikker på hva som blir best måte å løse det på.

Kan det være at vi burde ha noe annet der i steden for eksempe favicon eller lignende? Headern ble jo designet for vår gamle logo

olav223 commented 3 months ago

image image Er det menigen at det skal være forskjell på disse avhening av hvor stor de er?

emberal commented 3 months ago

image image Er det menigen at det skal være forskjell på disse avhening av hvor stor de er?

Det ble gjort av en grunn, men husker ikke helt hvorfor :thinking: Det ser helt fint ut med den hvite bak logoen på begge, så har reversert det.

simsine commented 3 months ago

LGTM 👍