responsive four cards feature using css grid

Solution retrospective
Just Completed my first challenge. Feedbacks would really be appreciated. let me know what and where i can improve. Thank You
Please log in to post a comment
Log in with GitHubCommunity feedback
- @osoriodev
Hello. 👋 Some things that could improve are:
- You are using 2
h1
tags, it is recommended to use only one tag for each page. (If you want to style the two title texts, you can put twospan
insideh1
) - Headings should increase steadily. You have an
h3
after theh1
, the correct thing to do would be to useh2
- You should use more semantic HTML (for example: article, section, etc.)
- And you have a typo in the line 8, you have opened the
link
tag twice.
Overall, your result was very accurate, congratulations.
Marked as helpful - You are using 2
- @mustysmalls
Noted will make corrections immediately and against my next challenge. Thanks Alot @Osoriodev I appreciate the feedback.
Join 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