single price grid

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 12 months ago

  Hello @falloumacbook!

  Your project looks great!

  Since you asked for feedback, I have a few suggestions for you to improve your project:

  • I noticed that you used margins to place your card closer to the middle of the page. There are 2 easier and very efficient ways to do it without margins or paddings.

  1. You can apply this to the body:

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

  or you can apply this to the element you wanna center:

  .element {
              position: absolute;
              top: 50%;
              left: 50%;
              transform: translate(-50%, -50%);
  }
  

  • Also, your box-shadow was applied only on the first container. Give the whole thing a box-shadow and it'll look even closer to the original design.

  I hope it helps!

  Other than those little details, you did a great job!

  Marked as helpful

  0

  ---

  

  Fallou Jeng•540

  @falloumacbookPosted 12 months ago

  Hello Daniel, thank you so much for your feedbacks, i didn't notice there was a box-shadow in the original design thanks alot for you comment imroving it right away.

  1

  

  Daniel 🛸•44,310

  @danielmrz-devPosted 12 months ago

  I'm glad I could help! 🙋🏽‍♂️ @falloumacbook

  0
• 

  Master•40

  @Fallou-JengPosted 12 months ago

  Nice work

  0