Design comparison
Solution retrospective
Your feedback is beneficial to me!
Community feedback
- @danielmrz-devPosted 12 months ago
Hello @AmitKumar-001
Your project looks great!
Since you asked for feedback, here are some tips for you to improve your project:
-
There's no need for that
h3
on top of your page. You can just remove it. -
Try wrapping your main content in a
<main>
tag instead of a<div>
tag. This makes your HTML more semantic. -
Every project should have a
<h1>
tag. We often wrap our main title with it. So yourh2
could be ah1
, since it's the most important text on the page. -
If you change your
background-color
to#D6E1F0
, your project will look even closer to the original design.
I hope it helps!
Other than those little details, you did a great job!
Marked as helpful1@AmitKumar-001Posted 12 months agoHello @danielmrz-dev
Thank you so much for your feedback. It is very helpful for me to improve my skills. I'm thankful for the suggestion.
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