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

Vanilla JS

#bem#fetch

@Lukasz-Milde

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


Feel free to point out all my mistakes.

Community feedback

Dytoma 570

@Dytoma

Posted

Hey👋

Good job on completing this challenge

For accessibility issues, I'll suggest you add the aria-label='next advice' attribute to your button and an aria-hidden="true" to the image inside the button as it just serves for decoration. You can read more about ARIA on mdn aria docs📚.

Happy coding🙌

Marked as helpful

1

@Lukasz-Milde

Posted

@Dytoma Awsome! Thanks!

1
Dytoma 570

@Dytoma

Posted

@Lukasz-Milde you're welcome🙌

0

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