Skip to content

Change header logo

Christoffer Hellerud Hansen requested to merge change-header-logo into master

Jeg fikk ikke til å inkludere kudaf dirigent logoen inni elementet slik at man får ikke en sånn onClick funksjonalitet som man kanskje hadde forventet på logoen også.

Jeg vet ikke om kanskje svg'en kan sendes ned i sds-header komponentet hvis de har noe logikk på at ting som sendes ned puttes i elementet. Jeg ser nå at logoElement her kunne inneholdt dirigent logoen våres, men akkurat nå så er logoElement enten en logotekst man sender inn som input til funksjonen, eller så er den en default Logo. Jeg kunne jo hørt med komponentbiblioteket om man burde hatt et input parameter til funksjonen som gjør at man kan få sin egen logo inkludert også

Ellers så kopierte jeg koden ganske rett av kunnskapsdata.no, jeg er på ingen måte en svg-animasjons-ekspert 😄 så kan hende det er noen rare ting i koden som jeg ikke plukket opp når jeg leste den/skrev den om

Siste ting: Jeg fjernet onKeyDown wrapper funksjonen som kalte på en click funksjon. Jeg var nysgjerrig på om den var nødvendig så jeg bare eksperimenterte med å fjerne den ett sted og det ser ikke ut til at den gjør noe lenger? Kan hende jeg ikke ser noe som du har sett, så bare si ifra hvis jeg skal fjerne den commiten 🙂

Merge request reports