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-component-card

@Tareeqcodes

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


Any feedback and corrections please.

Community feedback

@scriptyvijay

Posted

Well you built it very good but i can suggest you some points to improve your design skills -

  1. Perfume Word need word spacing
  2. Your main div isn't in center
  3. Need some padding in bottom after pricing
  4. Image is streching you can use object fit property to fix it.
  5. Description words are getting some spaces which not looking good

Hopefully this helps you.

Marked as helpful

0
Daniel 🛸 44,230

@danielmrz-dev

Posted

Hello @tariqauwal2020!

Your project looks great!

I noticed that you used margin to place the card in the middle of the page. Here's a very efficient (and better) way to center the card:

  • Apply this to the body (in order to work properly, don't use position or margins):
body {
    min-height: 100vh;
    display: flex;
    justify-content: center;
    align-items: center;
}

I hope it helps!

Other than that, great job!

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