Submitted over 3 years ago
Mobile first responsive 4 cards feature section
@gfontorbe
Design comparison
SolutionDesign
Community feedback
- @grace-snowPosted over 3 years ago
Hi
This looks really nice, but you need to change elements to make your html make semantic sense. Everything is a heading at the moment and that's not right.
You can only have one h1 per page too
Last thing to look up is how and when to write alt attributes. What you've got at the moment wouldn't be read correctly by a screenreader.
Good luck with it
0@gfontorbePosted over 3 years ago@grace-snow Thanks for the advice. I will pay more attention to those in my next submission.
Any reading you would recommend on how to organize HTML elements?
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