Design comparison
Solution retrospective
I had a firm grip of the application of css grid and flexbox in this project.
Community feedback
- @TedJenklerPosted 2 months ago
Hi @Jimztech,
Nice project! Here are a few suggestions for improvement:
Semantic HTML: Consider replacing <div> elements with more semantic HTML elements. For instance, use <main> instead of "main-content," and <article> or <section> instead of "cards." This change will improve both accessibility and SEO.
Heading Structure: Keep in mind that <h1> should be used only once per page to represent the main heading. For other sections, <h2> is preferable as the first heading. This helps maintain a clear and logical heading structure.
Keep up the great work!
Best, Teodor
Marked as helpful0 - @GhassanAlKaraanPosted 2 months ago
wow cool work.
the gap between the cards should be the same horizontally and vertically.
if you're using grid, it could be:
gap: 2rem;
Marked as helpful0
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