Flexbox Advice App

Community feedback

• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

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

  • add cursor pointer to btncontainer so that the user knows its clickable

  • since the btncontainer is interactive element it is better to wrap it inside a <button> tag instead of a div

  • check the HTML report to fix accessiblity issues

  hope this is helpful

  Marked as helpful

  1

  ---

  

  P

  Josh D•160

  @Josh-D18Posted over 2 years ago

  @Old1337 Thank you so much! I will add those changes!

  1
• 

  heba•320

  @hebam17Posted over 2 years ago

  Well done, I would suggest making the button a little bigger and decreasing spaces on the sides

  Marked as helpful

  1

  ---

  

  P

  Josh D•160

  @Josh-D18Posted over 2 years ago

  @hebam17 Thank you very much! I'll do that!

  0

  

  heba•320

  @hebam17Posted over 2 years ago

  @Josh-D18 You are welcome

  1