Product-preview-card-component Using HTML & CSS

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 8 months ago

  Hello @Mazen922!

  Your project looks great!

  I have one suggestion for you to improve it even more:

  • Using margin is not the best option to center an element. Here's a very efficient (and better) way to place an element in the middle of the page both vertically and horizontally:

  📌 Apply this to the body (in order to work properly, don't use position or margins):

  body {
      min-height: 100vh;
      display: flex;  /* it works with grid too  */
      justify-content: center;
      align-items: center;
  }
  

  I hope it helps!

  Other than that, great job!

  Marked as helpful

  0

  ---

  

  Mazen Ahmed•30

  @Mazen922Posted 8 months ago

  Thanks @danielmrz-dev

  it helped me a lot. 😊

  1
• 

  Luis Rivero•100

  @LJRiveroPastranaPosted 8 months ago

  well done! take a look of style guide and use the right tipography, that will look better the design.

  Marked as helpful

  0

  ---

  

  Mazen Ahmed•30

  @Mazen922Posted 8 months ago

  Hi @LJRiveroPastrana

  I did try to make the same font but I couldn't because it's not at Google Fonts these days, maybe it is too old and they deleted it.

  Thanks 😊

  0