P
ApplePieGiraffeā¢ 30,545
@ApplePieGiraffe
Posted
Good work!
I'd also add some padding to the left and right sides of the component so that there's some space between it and the edges of the viewport when you resize your screen.
š
2
Design comparison
Solution retrospective
I'd love some general pointers on how to improve my CSS.
Community feedback
Guy Ezraā¢ 615
@ezraguy
Posted
Looks good! , I would change 2 things...
-
I would change the media query to a
@media screen and (max-width: 700px)because as of now elements break on around 615px width like the "poster-name" and "poster-date". -
I would add
overflow-x: hiddento the body CSS to prevent the horizontal scroll.
2
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord