Order_Summary_HTML and CSS

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hi Bobby, great work on this challenge! 😉

  Here are a few tips for improve your code:

  • use main tag to wrap the card and improve the Accessibility, not for the inside card content
  • add descriptive text in the alt attribute of the image
  • remove all unnecessary code, the less you write the better as well as being clearer: for example the header tag container of image
  • remove all margin from .card-body class
  • use flexbox to the body to center the card. Read here -> best flex guide
  • add transition on the element with hover effect
  • instead of using px or % use relative units of measurement like rem -> read here

  Overall you did well 😁 Hope this help!

  Marked as helpful

  0
• Account deleted

  hi there,

  remove all the margin from the card div and give the body these properties to center the element display:flex justifiy-content:center align items: center min-height:100vh flex-direction:column

  • check the HTML report to fix accessiblity issues

  i hope this is helpful and goodluck!

  Marked as helpful

  0