Advice generator app

Community feedback

• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 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 👍

  Marked as helpful

  1

  ---

  

  Azeem Saifi•370

  @saifion33Posted over 2 years ago

  @TheCoderGuru Hi coder guru. Thanks for your feedback

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 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😀

  1

  ---

  

  Azeem Saifi•370

  @saifion33Posted over 2 years ago

  @Crazimonk Hi Naveen . Thanks for your feedback

  1