Skip to content
Submitted almost 2 years ago

Vanilla JS

Lukasz-Milde•150
@Lukasz-Milde
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Advice generator app
  • HTML
  • CSS
  • JS
  • API
2junior

Design comparison

SolutionDesign

Solution retrospective

Feel free to point out all my mistakes.

Community feedback

  • Dytoma•550
    @DytomaPosted almost 2 years ago

    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•150
    @Lukasz-MildePosted almost 2 years ago

    @Dytoma Awsome! Thanks!

    1
    Dytoma•550
    @DytomaPosted almost 2 years ago

    @Lukasz-Milde you're welcome🙌

    0

Please log in to post a comment

Log in with GitHub

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