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

Product preview card component

@msyahreza

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

@nenadmne

Posted

Hello, i had some spare time so i look into your code and shere few thoughts.

h1 { font-weight: bold; }

// I would use just font-weight: 700; as you did in .category further bellow. //

@media only screen and (max-width: 700px) { .prices { display: flex; align-items: center; }

Here i saw you repeated some of the atributtes from previus code. When using @media you should write down only code that is changing further.

Last thing i would suggest is that instead writting for example display:flex, for every class, you just tight them all under same line:

.one, .two, .three, .four { display:flex; }

nice project and happy coding =)

1

@msyahreza

Posted

@nenadmne thanks for your corrections 😄

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