QR-code-Project

Community feedback

• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

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

  Lets firstly work on your accessibility issues.

  • Document should have on main landmark basically means your html should be structured more semantically and the correct format should be your <header>......</header> followed by your <main>......</main> and lastly your <footer>....</footer> hence you should use <main class="container"> instead of <div class="container">.
  • Page should contain a level-one heading basically means your html should have a h1 it aid navigation hence <h2>Improve your front-end <br> skills by building projects</h2> should be <h1>Improve your front-end<br> skills by building projects</h1> and you should also go down orderly when you are using the headings h1 down to h2 down to h3 and so on.
  • Images must have alternate text - Its hard for screen readers to pick up messages from images without alt text.. Its always advisable to include alt text to aid screen reader

  This should fix most of your accessibility issues.

  • You should increase your border-radius to make it more curvy.
  • You should also use rem em % instead of px when measuring.. It improves responsiveness

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

  NAIJA TO THE WORLD!!!

  Marked as helpful

  0

  ---

  

  PRECIOUS ORITSEDERE•120

  @PreciousOritsederePosted over 2 years ago

  @Samadeen Thank you. I'll look into these

  0

  

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  @PreciousOritsedere Oil dey your Head

  0

  

  PRECIOUS ORITSEDERE•120

  @PreciousOritsederePosted over 2 years ago

  @Samadeen Haha thanks

  0