Design comparison
SolutionDesign
Community feedback
- @beniusisPosted 10 months ago
Hey 👋 Good job on completing the challenge!
A few things worth mentioning:
- You should use only one
<h1>
tag per page. You can read more about the accessible heading structure here. - You should not leave empty
alt
attributes for images. It is read by screen readers for people with visual impairments and is better for SEO. You can read more about it here. - You could use semantic elements like
<main>
,<footer>
,<header>
to improve accessibility and achieve better organization. For example, switch<div class="containt">
to<main class="containt">
.
I hope the feedback was helpful. Happy coding! 😀
Marked as helpful0@itsU-KPosted 10 months agoThank you for your feedback @beniusis it's really helpful I'll implement it from now👍🏻
0 - You should use only one
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