Advice generator

Community feedback

• 

  pascal de creator•120

  @pascall-de-creatorPosted over 2 years ago

  Hi Prince,

  Great job.. its really nice... :{D

  I might have a solution to your problem but it will be hard to explain... I will have to clone your repo and give it a fix .... I will also leave some fix notes so you will know the places I fixed and the issues :D

  Marked as helpful

  1
• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey princej02, good job completing this challenge. Keep up the good work.

  Your design looks good however I think that your card background is supposed to be a bit darker in addition the font size of the card title should be a bit smaller.

  In terms of your accessibility issues

  • Buttons need text in order to be clickable, you can achieve this using the aria-labelledby attribute.

  • The object tag requires an alternative description. You can fix this by specifying the alt attribute with a description of the image.

  • Wrap all your content in <main> tag to fix all other accessibility issues.

  I hope this helps

  Cheers, Happy coding 👍

  Marked as helpful

  0
• 

  Prince•210

  @princej02Posted over 2 years ago

  Okay, thanks.

  0