Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

html & css, flexbox

Saleiux 250

@Saleiux

Desktop design screenshot for the Blog preview card coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@MelvinAguilar

Posted

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 250

@Saleiux

Posted

Thankyou for your suggestions!@MelvinAguilar

1

Please log in to post a comment

Log in with GitHub
Discord logo

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