Design comparison
Solution retrospective
Feedbacks are welcomed. I couldn't make it mobile responsive.
Community feedback
- @Harsh-Kumar-DwivediPosted 8 months ago
Hi @riaz37,
I would suggest following:-
1.Prefer replacing
<div class="container">
with<main>
element.2.Headings are required to be in a hierarchical order.
3.Prefer giving more descriptive name to classes so that it is easier for you or someone who tries to understand your code gets it smoothly.
4.Understanding of flexbox would be helpful.
These will enhance your code, improve accessibility and make your markup more semantic.
Hope these suggestions are helpful !
Happy Coding !
Marked as helpful0@riaz37Posted 8 months ago@Harsh-Kumar-Dwivedi
Thank you very much for your suggestions. I will try my best.
1
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