Altinn / altinn-cdn

Static assets used by Altinn solutions and apps.
https://altinncdn.no
1 stars 30 forks source link

Endret default logo for SSB #143

Closed StianVestli closed 11 months ago

StianVestli commented 11 months ago

Description

Endret logo til ssb_dark.png som er logo med tekst. Endret altinn-orgs.json , default logo til ssb_dark.png

altinnadmin commented 11 months ago

@StianVestli Ser bra ut. Skal jeg bare merge slik at oppdateringene deployes til CDN? Antar du har testet med denne logoen i en app for å verifisere at det ser bra ut? :)

StianVestli commented 11 months ago

Det burde je sikker ha gjort, men får jeg testa den lokalt da i den plasseringen jeg vil ha?

altinnadmin commented 11 months ago

Det burde je sikker ha gjort, men får jeg testa den lokalt da i den plasseringen jeg vil ha?

Yup, skal være å bare legge inn en URL som en tekstressurs, og så peke på den, ref. https://docs.altinn.studio/nb/app/development/ux/organisation-logo/#source

image

StianVestli commented 11 months ago

Ja jeg kan peke på en fil som er publisert men får ikke det mot min fork.

altinnadmin commented 11 months ago

Fungerer ikke denne lenken til bildet fra PR'en som du lagde? https://raw.githubusercontent.com/Altinn/altinn-cdn/7dff0c3bcf6713293bf31b95c54a05e81334dd32/orgs/ssb/ssb_dark.png

StianVestli commented 11 months ago

Den lenka der funket.. men jeg finner ingen url som ser slik ut hos meg.. da skal je teste

StianVestli commented 11 months ago

Hmm ser ut til at logoen blir veldig liten... så spørs om jeg har snubla i noe det samme som Finanstilsynet hadde. for logoen i seg sjølv burde jo funke greit.

altinnadmin commented 11 months ago

ser ut til at logoen blir veldig liten

Er kanskje litt mye "luft" i selve logoen? Selve logoen er jo bare 1/3 av total høyde... image

StianVestli commented 11 months ago

Jeg skal se på logoen og opprette ny PR...