Design comparison
SolutionDesign
Solution retrospective
Feedback is very welcome!
Community feedback
- @MelvinAguilarPosted 12 months ago
Hello there ๐. Good job on completing the challenge !
I have other suggestions about your code that might interest you.
- If the image is decorative and doesn't convey meaningful content to the page, you can leave its alt attribute empty. This helps screen readers skip unnecessary information. Additionally, the alt attribute should describe the image, avoiding redundant words like "image", 'picture", or "photo" since the
<img>
tag already conveys that information. For a photo of a person, using their name as the alt text is a good practice.
- You should use only one
<h1>
tag per page. The<h1>
tag is the most important heading tag, This can confuse screen reader users and search engines. You can read more about this here ๐.
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
Marked as helpful0 - If the image is decorative and doesn't convey meaningful content to the page, you can leave its alt attribute empty. This helps screen readers skip unnecessary information. Additionally, the alt attribute should describe the image, avoiding redundant words like "image", 'picture", or "photo" since the
- @AdrianMajPosted 12 months ago
I think everything looks good there, it is accessible and responsive. You are using semantic HTML, layout looks good on phones. I have no reservations for that. Thanks!
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