Fully responsive webpage using css grid

Community feedback

• 

  P

  Ken•4,915

  @kens-visuals

  Posted over 3 years ago

  Hey @Theguydev 👋🏻

  I have some suggestions to help you fix the accessibility issues and some other things.

  • In your markup, <div class="container">...</div> should be <main class="container">...</main> and <div class="header-container">...</div> should be <header class="header-container">...</header>. These will fix the accessibility issues. Don't forget to generate a new repot once you fix the issues.
  • For the icons, add aria-hidden="true”, because they are for decoration. You can read more about aria-hidden here. For example:

  <img src="assets/icon-supervisor.svg" alt="" aria-hidden="true”>
  

  • Next, <p class="main">...</p> should have text-align: center;, especially on mobile viewport width.
  • I won't go into many details about resetting CSS, but I'll leave this cool article here, which will make more sense than my brief explanation.

  I hope this was helpful 👨🏻‍💻 all in all, you did a good job with the project, keep it up. Cheers 👾

  10

  Marked as helpful