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

Flexbox, Figma

Radek 490

@Radexman

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

MURRAY122 280

@MURRAY122

Posted

Hi Radek,

Congrats on your first design! I took a quick look, and it seems a bit large for the screen sizes. Seems like there is a lot of set margins and padding for elements. You might want to consider the following to start with:

body{
    display: flex;
    align-items: center;
    justify-content: center;
    height: 100vh;
    width: 100vw;
}

You could then remove the width and margin styles on main to center the element. Then you could set the img to height: 450px and remove its width. From there it would be about changing the products description and title font sizing, padding and margins etc...

  • You may also be interested in media query to help support responsive design and answer any question you might have had with it.

Marked as helpful

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