
Design comparison
Community feedback
- @DanieleErcoli243Posted 3 months ago
Hi. I read the code and actually the HTML is semantic. I liked the use of the header tag for the upper part of the page, but I've been taught no to use many h1. A page should have just one h1 and then the other headings are supposed to go from h2. I really envy the use of root properties. I saw that the padding between the cards is not regular. You'd better use a system of row and cols, giving some padding to the cols and a negative margin to the row.
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