Skip to content
Submitted about 1 month ago

Article preview component

P
AEO•280
@Jayfx24
A solution to the Article preview component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

What are you most proud of, and what would you do differently next time?

Getting a bit better using media query, i know more now than the last project.

Code

Please log in to post a comment

Log in

Community feedback

  • Dorm-24•270
    @Dorm-24

    Hi AEO, I can't give you a good advice because I myself can't work well with media queries, however I can note some things I've noticed when I went through your code. Firstly, I don't really get why you made all the spacings in custom properties and I think that isn't necessary. Secondly, in the media query: article { display: grid; grid-template-rows: 40% 1fr; height: 60vh ; }, I made it instead of using grid, I used flex-direction: column, and in this way it is way better looking and more responsive. There are a few more small things I could mention but I don't think it will be necessary. Everything else looks good enough!

    Marked as helpful

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