Advice generator using advice slip JSON API

Community feedback

• 

  Jordane Gengo•590

  @jgengo-alt

  Posted about 3 years ago

  Hello there 👋 I'm back to give some advice on your latest assignment :)

  I find interesting that you made yourself the -- " -- under the advices instead of using an image, you gain speed in loading the page, that's awesome!

  However, what you gain to load the page, you sadly lost it by reloading the page everytime we click the button. Would have been better to make onclick call a function that would have fetch a new advice and replace only the needed text on the page instead of reloading everything.

  Also with a wide screen, your main div would have benefits having a max-width ;)

  Keep going 💪

  01

  Marked as helpful
• Account deletedPosted about 3 years ago

  Hi there,

  • put the quote inside a <blockquote> tag instead of h1 to be more semantic

  • check Html report to fix accessibility issues

  i hope this is helpful and goodluck!

  01

  Marked as helpful