Order Summary Component | HTML CSS

Community feedback

• 

  Rachel Cessna•195

  @cessnar516Posted about 3 years ago

  The overall layout looks like it's mostly there - just needs a few tweaks to get it to match the example and take care of the HTML and accessibility issues highlighted in the report for your solution. Here are a few suggestions:

  • It looks like like you're missing the Google font called for in the style guide, so you'll need to add a link to that stylesheet in your HTML head and set the font-family for the body.
  • Add an empty alt attribute to all of the images since they are decorative: alt="" .
  • Try using semantic HTML elements to define the sections of the page. I suggest at least adding a <main> inside of the body element and then adding a <footer> element to contain the .attribution information. You could change your .order-card div to a <section> element too and replace your current sections inside of the .change-box with divs instead because section elements require a heading.

  Hope this helps!

  0
• 

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  Take a look on your HTML and accessibility issues. Also, add background color to body and make attribution outside of the main card

  0