Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Advice Generator App Flexbox, and Vanilla JavaScript

P
Eileen dangeloโ€ข 1,600

@Eileenpk

Desktop design screenshot for the Advice generator app coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
  • API
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


This was my first "real" project using an API. Any comments or feedback is always appreciated :)

Community feedback

P
ApplePieGiraffeโ€ข 30,565

@ApplePieGiraffe

Posted

Hello there, Eileen dangelo! ๐Ÿ‘‹

Nice to see you complete another challenge! ๐Ÿ˜€ Well done on this one! ๐Ÿ‘

One small suggestion I have is to add some screenreader-only text to the inside of the button or add an aria-label instead that describes the purpose of the button. This is important for accessibility reasons because there is currently nothing that screen readers can read to let users know what that button is for. If you'd like to learn more about creating elements that are visually hidden but can still be read by assistive technologies, check out this nice article.

Hope that helps. ๐Ÿ˜Š

Keep coding (and happy coding, too)! ๐Ÿ˜

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord