Product preview card

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace the <h2> with <p> and <h3> with <h2> to fix the accessibility and warning issues.

  Use the colors that were given in the stylguide.md found in the zip folder of the challenge.

  add box-sizing border-box to the *{} or the body , the box-sizing -border box deduct the padding value from the width value instead adding it to the width value.

  Give the main a fixed width value: e.g: main{width: 500px} Give the margins, width, paddings and border-radius a fixed value instead of a percentage value. Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To

  you forgot to add a media query for the mobile design.

  Hope am helpful. Happy coding

  Marked as helpful

  1