Design comparison
Solution retrospective
Hey guys, if you have any tips or advices are welcome. Happy coding :)
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi Paul, how are you?
I really liked the result of your project, but I have some tips that I think you will like:
1- Heading levels should only increase by one click here
I noticed that you used an
a
ie an anchor tag, the most recommended would be thebutton
element in addition to making the code more semantic. To improve even more you could put anaria-label
explaining what the button's function is.The rest is really good! Congratulations on the evolution. Hope it helps... 👍
Marked as helpful1 - @vanzasetiaPosted about 2 years ago
Hello, Paulo Henriques! 👋
Good effort on this challenge! 👍
Headings must be in a logical order. Users of assistive technology can use heading tags to navigate the website. If headings are not in a logical order, those users might get confused. I highly recommend reading the "How-to: Accessible heading structure - The A11Y Project" article.
I hope this helps! Happy coding! 😄
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin 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