Order-summary-component

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Wrap <div class="attribution"> within the footer tag to fix the accessibility issue.

  Change the background-size of the main to contain, instead of cover. For a responsive content, replace the width of .card with max-width and give it a fixed value.

  You forgot. give .proceed a box shadow value. give .plan a background-color and a border-radius, in place of a justify-content: space-between use gap and give it a value gap:10px; then you give.

  There is no need to give . summary a width value , give tit a pladding value for all the sides and a margin-bottom value.

  The hover of the cancel order is a darker font-color. you forgot to add a media query to the design , in the media query you only have to change the background-image of the main

  Use rem and em as unit for the width, margin, padding and font size values instead px which is an absolute unit. For more on CSS units click here

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  0