Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make box-shadow be a part of element width for embeds #2423

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

katrinewi
Copy link
Contributor

Dette er en fiks for å tvinge boxShadow: "full" til å ta opp faktisk plass i embeds.

Hentet inspirasjon herfra: https://stackoverflow.com/a/14367262

Syntes dette ser ut til å fungere, prøvde å gjøre tilsvarende for height, men det hadde ingen effekt, som jeg syntes var litt rart ...

Eventuelt er en mulig løsning å lage en wrapper som setter padding tilsvarende plassen box-shadow opptar. Prøvde meg også litt frem med pseudo-element, men det vil ikke fungere med padding utenfra, som er litt av poenget her 😅

@katrinewi katrinewi requested review from Jonas-C and a team August 15, 2024 07:25
@Jonas-C
Copy link
Contributor

Jonas-C commented Aug 15, 2024

Fint dette altså! Men hadde vært veldig greit å få det til for høyde også. Det er tross alt der margin i artikkel faktisk er et ordentlig problem mtp box-shadow :')

@katrinewi
Copy link
Contributor Author

Fint dette altså! Men hadde vært veldig greit å få det til for høyde også. Det er tross alt der margin i artikkel faktisk er et ordentlig problem mtp box-shadow :')

Får se litt mer på hvorfor det ikke applyes der! forsåvidt et lite problem at embeds var "utenfor" artikkel-paddingen og da

@katrinewi katrinewi marked this pull request as draft August 15, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants