Design comparison
Solution retrospective
Pointing out flaws welcome :)
Community feedback
- @blockfl0wPosted over 2 years ago
The
<article>
element is misused in this project the element should be used to show the entierty of a component. I would recomend that you switch all your<article>
tags to<section>
to help screenreaders determine what your trying to tell the user. For more info on the<article>
tag this mdn doc should help youyou also dont use a "landmark element" which includes elements like main, nav , footer, etc This helps screenreaders know the purpose of your website and understand the structure of it. It also helps with SEO I would recomend that you switch your div with the class "container-container" to a
<main>
element.This mdn doc will explain it better than me.
and this mdn doc will explain website structure and all the "landmark elements".
I hope this helps you!
0
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