AtB-AS / webshop

AtB Webshop / AtB Nettbutikk

https://nettbutikk.atb.no

European Union Public License 1.2

0 stars 1 forks source link

fix: svg loading from css files #473

Closed mikaelbr closed 2 years ago

mikaelbr commented 2 years ago

Virker som webpack her setter opp noe proxy-filer for routing som ikke blir håndtert rett. Men etter nærmere ettertanke så tenker jeg at det er vel ingen grunn til å ha url eller file loading på data fra css/scss her. Dette unngår å gjøre noen ekstra kode for håndtering av svgs fra css og fikser feil med ikke lastet bakgrunnsbilde nå i staging og prod.

Testplan staging

[ ] Bakgrunn skal vises i dark og light mode (for NFK og ATB)
[ ] tkort illustrasjon i onboarding skal vises komplett.

github-actions[bot] commented 2 years ago

Visit the preview URL for this PR (updated for commit 8a434df):

https://atb-webshop-staging--pr473-mikael-fix-svg-css-yrq7rlkg.web.app

_{(expires Wed, 09 Feb 2022 12:46:51 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

mikaelbr commented 2 years ago

@tormoseng Det har vært en feil i nettbutikk prod/staging ang tkort og bakgrunn. Denne skal fikse det. la til testscenario for kontekst i beskrivelse.

tormoseng commented 2 years ago

Testet OK for AtB.

@mikaelbr Tester dere manuelt for NFK også?

mikaelbr commented 2 years ago

Ja, det gjør vi!

tormoseng commented 2 years ago

@mikaelbr En liten kommentar. Husker ikke om dette alltid har vært slik, men la merke til at AtB-logo øverst på fanen blir veldig mørk i dark mode. Hvis ikke denne er endret nå, skal jeg gå andre steder.

Mulig dette er bare mitt problem som har 30+ faner åpne, hvor da kun ikonet vises..

mikaelbr commented 2 years ago

Jeg mener den har alltid vært slik og jeg har registrert det samme, men ikke tatt meg tid til å fikse. Men det er fint mulig å fikse. Hos NFK "fikset" vi det bare med å bruke en annen farge.

tormoseng commented 2 years ago

Mm. Du har nok rett. Sender denne saken videre, og gir en notis til @hildeor ☝️