Daniel 🛸• 44,230
@danielmrz-dev
Posted
Hello @imabhijeet!
Your project looks great!
I have a few tips for you to improve it:
- The card has curvy edges, so you can add
border-radiusto get that result; - You can check the
style-guidefile to check the recommendedfont-familyto the project. By the way, the font you used was a good one for this project. - The button Learn more is a clickable element, so it's nice to add
cursor: pointerto it. - Lastly, you can replace your
div.containerwithmain.container. This won't change anything visually, but it makes your HTML code more semantic and improves SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than those little things, you did an excelent job!
Marked as helpful
1
abhijeet• 30
@imabhijeet
Posted
Thankyou @danielmrz-dev for your valuable time. Some of the changes were intentional. Although I really appreciate that you pointed out the lack of SEO optimization of my project, I have implemented all the necessary changes that you have mentioned.
1
Daniel 🛸• 44,230
@danielmrz-dev
Posted
I'm glad my feedback was helpful 😊 @imabhijeet
Marked as helpful
1
