Project-insert-name / root-website

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

Change css on images #126

Closed olav223 closed 7 months ago

olav223 commented 7 months ago

Mister litt design men du får hele bilde inn i rammen uten at den er strukket. https://root-website-git-125-bilde-cropping-pin-root-webside.vercel.app?_vercel_share=FKR0HMK7Q47w3UHrL68oJwck3nBobtJO Før: image

Etter: image

annet forslag:

image

Legg til "border-1 border-root-primary-dark" på linje 24 i fil ./components/cards/thumbnail.tsx

Før: image

Etter: image

vercel[bot] commented 7 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 Feb 12, 2024 4:07pm
emberal commented 7 months ago

Det er nok det vi må gjøre hvis vi alltid skal vise hele bilde. Synes ikke det ser så bra ut, men det hvis bildene skal inneholde viktig info så må vi nesten gjøre noe sånt.

En annen mulighet kan være å fjerne makshøyden på bilde, men det kan føre til at bilde tar opp hele høyden på skjermen.

Tror jeg ville behold thumbnail bildene som de var tidligere da, med object-cover. Siden det er ikke så viktig å vise hele bildet der, når man ikke klarer å lese det med liten tekst uansett.

emberal commented 7 months ago

En annen mulighet kan være å gjøre litt som Facebook, ved å kutte kun bunnen av bilde.

image

Og etter du trykker på så, ser man hele bilde

olav223 commented 7 months ago

En annen mulighet kan være å gjøre litt som Facebook, ved å kutte kun bunnen av bilde.

image

Og etter du trykker på så, ser man hele bilde

Det kan jo være mulig

emberal commented 7 months ago

Veldig usikker på dette fortsatt, men ser fordelen med å vise hele bilde. Prøvde å legge til en avrundet kant rundt bilde også, men var visst ikke så lett.

olav223 commented 7 months ago

Det funker slik det er foreløpig