Design comparison
SolutionDesign
Community feedback
- @FluffyKasPosted about 3 years ago
Hi, it looks really very good! Your CSS looks very neat and easy to understand, I'd only argue about your html. Not sure, if using sections here is appropriate. Also, heading should only increase by 1. And instead of a div (wrapper) you could perhaps use
main
. There's also no need to add a class name for thebody
as there's only one in a html document.Marked as helpful1@CyrusKabirPosted about 3 years ago@FluffyKas hello hello thank you for your feedback but i change right now i mean 2 minute ago that div and main problem and i forget use landmarks but thank for your help to improve myself (sorry if my english grammer is bad)
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