Skip to content

Redesign#162

Merged
simsine merged 23 commits intomainfrom
115-bytte-til-ny-logo-og-erstatt-stilen-slik-at-det-passer-med-logoen
Jun 8, 2024
Merged

Redesign#162
simsine merged 23 commits intomainfrom
115-bytte-til-ny-logo-og-erstatt-stilen-slik-at-det-passer-med-logoen

Conversation

@simsine
Copy link
Copy Markdown
Member

@simsine simsine commented Mar 7, 2024

Beskriv endringene

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

Relaterte issues

@simsine simsine added the enhancement New feature or request label Mar 7, 2024
@simsine simsine linked an issue Mar 7, 2024 that may be closed by this pull request
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 7, 2024

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

…-og-erstatt-stilen-slik-at-det-passer-med-logoen

# Conflicts:
#	pnpm-lock.yaml
emberal added 5 commits May 15, 2024 10:17
Fikset pnpm-lock var utdatert og lagt inn dependency på sanity/ui
Liten justering på logo, små skjermer.

Fikset farge på lenke i footer
@simsine simsine requested review from emberal and olav223 June 3, 2024 19:44
@simsine
Copy link
Copy Markdown
Member Author

simsine commented Jun 3, 2024

@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 👍

@olav223 olav223 marked this pull request as ready for review June 3, 2024 19:53
@olav223
Copy link
Copy Markdown
Contributor

olav223 commented Jun 3, 2024

Er det meningen logoen skal se slik ut?
image
image

@olav223
Copy link
Copy Markdown
Contributor

olav223 commented Jun 3, 2024

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

@simsine
Copy link
Copy Markdown
Member Author

simsine commented Jun 3, 2024

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
Copy link
Copy Markdown
Member Author

simsine commented Jun 3, 2024

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
Copy link
Copy Markdown
Contributor

emberal commented Jun 3, 2024

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
Copy link
Copy Markdown
Contributor

olav223 commented Jun 3, 2024

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
Copy link
Copy Markdown
Contributor

olav223 commented Jun 6, 2024

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

@emberal
Copy link
Copy Markdown
Contributor

emberal commented Jun 8, 2024

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 🤔 Det ser helt fint ut med den hvite bak logoen på begge, så har reversert det.

Copy link
Copy Markdown
Contributor

@olav223 olav223 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@simsine Du kan squash and merge for min del.

@simsine
Copy link
Copy Markdown
Member Author

simsine commented Jun 8, 2024

LGTM 👍

@simsine simsine merged commit a414bea into main Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dark mode Discord lenke i footer Bytte til ny logo, og erstatt stilen slik at det passer med logoen.

3 participants