NFT preview card component

Community feedback

• 

  Omar M.•270

  @OmarMAttia7Posted almost 3 years ago

  Congratulations on finishing the challenge :). You have some problems with HTML markup, most of them are in the report:

  1. All images must have a meaningful alt attribute that describes the image for people who can't view the image visually.
  2. The main content of your page should be inside a <main> element, it's preferable that you give it the class="container" instead and remove the <div class="container"> as it has no use.
  3. [the footer element should be inside the body element] (https://teamtreehouse.com/community/why-is-the-footer-placed-before-the-closing-of-the-body-tag-and-not-after).
  4. Any HTML document should have an <h1> heading element, in your case you should put <a class='title'>Equilibrium #3429</a> inside an <h1>.

  Good luck and keep coding <3.

  Marked as helpful

  1

  ---

  

  RonanC•30

  @RonanCaPosted almost 3 years ago

  @hazel79 Thank you very much for your feedback Omar, this is very helpful !

  1