QR-code-component

Community feedback

• 

  Hassia Issah•50,390

  @Hassiai

  Posted over 2 years ago

  add the alt attribute alt=" " to the img tag and <h3> with <h1> to fix the accessibility and error issues. the value of the alt is the description of the image.

  For a responsive content which wont require a media query for this challenge, replace the width of .cart with max-width and give it a fixed value. e.g: max-width:350px;. There is no need for a height value .cart , the padding value can be a replacement of that and it a responsive replacement.

  Give the img a max-width of 100% instead of a width height value , there is no need for a margin value in the img.

  There is no need for margin value in .text, increase the padding vale and add a font-size of 15px to it , this will reduce the font-size of h1 and p.

  Use relative units like rem or em as unit for the padding, margin, width values and preferably rem for the font-size values, instead of using px which is an absolute unit. For more on CSS units Click here

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  00