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

Flex-box, media query

@Pepeleeno

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


He Everyone, this is my first challenge I completed on this website. Don't have much to say. I found it easy. Spent about an hour for this project

Community feedback

P. Ricardo 2,370

@pRicard0

Posted

You should never use margin to position this way. I recommend using flexbox in the body. The body needs min-height: 100vh to perfectly center the content using flexbox. You can learn more about flexbox in an interactive way by clicking here.

Your project must contain a main instead of a div with the "container" class. You can use section instead of using divs with the section classes.

You shouldn't use px for font size, use em or rem.

Images must contain a short description in the alt attribute.

1

@Pepeleeno

Posted

@pRicard0 Thank you for the feedback. I'll definitely correct my mistakes.

0

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