@caarlosdamian
Posted
Congratulations on completing the challenge! 🎉 I have some recommendations for your code that I think you will find useful.
First, let's talk about HTML 🏷. Your solution currently generates accessibility error reports, and one of the issues is that "All page content should be contained by landmarks" due to non-semantic markup. To improve the accessibility and organization of your page, I suggest wrapping the entire content (which includes multiple section elements) with the semantic element <main> in your index.html file.
You may be wondering, what is meant by a landmark? Landmarks are used to define major sections of your page instead of relying on generic elements like <div> or <span> . They convey the structure of your page. For example, the <main> element should include all content directly related to the page's main idea, so there should only be one per page.
I hope you find this helpful 😊 Your solution is great, and I wish you all the best in your coding journey!
Happy coding! 🚀
Marked as helpful
