Skip to content
Submitted over 2 years ago

Aplicación generadora de consejos

Manuel M. Pilar•30
@Manuel23pro
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

esta pequeña app me ayudo a reforzar mi conocimiento de javaScript

Community feedback

  • Nick•110
    @Fitsos17Posted over 2 years ago

    Hi man. Nice solution. I just want to help you fix those accessibility and HTML issues.

    • Don't forget to add an alt in all of your img tags. It helps for screen readers.
    • The error All page content should be contained by landmarks can be solved by wrapping the whole app in a <main></main> tag.
    • Page should contain a level-one heading. This says that you should add an <h1></h1> tag. Try replacing every span and p tags with headings from h1 to h6.

    These should fix the issues. Very nice solution. Keep it up, Nick.

    0
    Manuel M. Pilar•30
    @Manuel23proPosted over 2 years ago

    @Fitsos17 Thank you very much for the help

    1

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