Design comparison
Solution retrospective
any review will surely be accepted..
Community feedback
- @DavidMorgadePosted about 2 years ago
Hello Manish congrats on finishing the challenge! you did a great job, your desing looks almost the same as the solution!
I have some advices for your html, instead of having your content on a
div
inside of themain
, I would definitely put all the content directly on themain
, and then divide it on sections, instead of using flexbox to center the whole card onmain
you can use it on yourbody
, then themain
will be the card, and you divide it in twosections
You can also use anchors
a
tags in your links for social media, whenever you click on something and it takes you to other page, thats an anchor tag and you should use it!Hope my feedback helped you! good job!
Marked as helpful0@manishdevelopsPosted about 2 years agothank you David ... your feedback helped me , I will look on it..
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