Order Summary

Community feedback

• 

  dewslyse•3,025

  @dewslysePosted over 3 years ago

  Hello Alicia 👋! Congrats on completing your first challenge on frontend mentor. Your solution is well implemented. Well done 👏👏👏. Some things to note:

  • You could wrap your img on line 20 of the html in a header or figure tag to have it selected by a landmark. For improved accessibility, add and empty alt (alt=" ") to the img tag and set aria-hidden: true.

  Alternatively, your could attach the image to the page in body of your css. Eg.:

  body {
     background: $pale-blue url("./images/pattern-background-desktop.svg") no-repeat;
     background-size: contain;
  }
  

  • h and p elements cannot be child elements of a span. You could replace the <span> with a <div>

  Happy coding and best of luck with subsequent challenges

  Marked as helpful

  0