Skip to content
Submitted 11 months ago

html & css, flexbox

Saleiux•230
@Saleiux
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Blog preview card
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I wasn’t sure how to center my elements in the middle of the page, did I do it the right way?

My HTML can be improved?

Did I use css best practices?

Community feedback

  • Melvin Aguilar 🧑🏻‍💻•61,140
    @MelvinAguilarPosted 11 months ago

    Good job on completing the challenge !

    I have some suggestions about your code that might interest you.

    • Use semantic elements such as <main> and <footer> to improve accessibility and organization of your page.
    • The issue I see is that the height of the .page element is too small; you should use min-height: 100vh to cover 100% of the device height and remove the height: 800px;. Additionally, consider adding justify-content: center in the .page element to vertically center the element instead of using margin-top: 180px.

    Happy coding!

    Marked as helpful
    0
    Saleiux•230
    @SaleiuxPosted 11 months ago

    Thankyou for your suggestions!@MelvinAguilar

    1

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