Responsive site with CSS Grid

Community feedback

• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Hey!! Cheers 🥂 on completing this challenge.. .

  Here are my suggestions..

  • You should use <main class="container"> instead of <div class="container">.
  • Go down orderly when you are using the headings h1 down to h2 down to h3 and so on.
  • You can wrap your attribution section in a footer tag to avoid accessibility issues.
  • You should remove your p nested in your span

  This should fix most of your accessibility issues and HTML

  . Regardless you did amazing... hope you find this useful... Happy coding!!!

  Marked as helpful

  0
• 

  Bheki Accion Nkomo•230

  @BhekiAccionPosted over 2 years ago

  Hey Abdul, thanks for the feedback, l really appreciate it. Yeah normally wrap the content inside the main tag l just forgot on this one, and ohh thanks l always wondered where l can put the attribution because l don't really want them in the main tag. Thank you so much for this feedback. l will grow from this.

  0