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

Css flex and Css grid

Vadym 30

@LLUXER

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


I like it very much, but please don't pay attention to the fact that I didn't optimize for other screens, I don't know how to do it and it's difficult, but thanks if you write what flaws you saw

Community feedback

@Alord81

Posted

Hi @LLUXER,

I think I can help you with the flaws I saw :

You can make the main tag <main> contain all the elements instead of making them dependent on the body directly if all elements belong to the same section.

Also if you want to optimize for other screens you can search for CSS Media Queries.

If you want to make the box in the middle of the page you can add min-height:100vh; display: flex; align-items: center: justify-content: center; to the body.

HOPE I COULD BE OF HELP :)

HAPPY CODING!

Marked as helpful

1

Vadym 30

@LLUXER

Posted

@Alord81 Thanks for the advice and help

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