Design comparison
SolutionDesign
Community feedback
- @brukmgPosted about 1 month ago
You’re already doing great by using semantic tags! Just a couple of tweaks, you don’t need to use role="main" and role="contentinfo" here, because main and footer are already semantic HTML5 tags that automatically serve those roles. Also, to make your code cleaner and easier to manage, it’s a good idea to store your colors in CSS variables. That way, you won’t have to repeat the same color values throughout your styles.
Marked as helpful0
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