Advice generator app

Community feedback

• 

  Shashree Samuel•8,860

  @shashreesamuel

  Posted over 3 years ago

  Good job with this challenge, keep up the good work.

  In terms of your accessibility issues, wrap all your content within a <main> tag.

  Your design looks good but I believe that the title where the advice number will be dynamically generated should be in a different font. Refer to style-guide in the starter files and change it.

  Overally, everything looks good.

  I hope this helps

  Cheers, Happy coding 👍

  11

  Marked as helpful
• 

  Naveen Gumaste•10,420

  @NaveenGumaste

  Posted over 3 years ago

  Hello saifion33 ! Congo 👏 on completing this challenge

  Let's look at some of your issues, shall we:

  • here <div id="advice-id">ADVICE #1</div> you have to use heading like h1

  happy Coding😀

  11