Submitted over 2 years ago
3 Column Preview Card | HTML CSS BEM MobileFirst
@Carlosgnx
Design comparison
SolutionDesign
Solution retrospective
Leave your feedback :D
Community feedback
- @AdanSaavedraPosted over 2 years ago
Well done bro, but i saw you forgot to put the hover state elements, you can learn more about it here. Also you have problems with the accessibility, I took a look to your code and you should use h1 tag first instead of h2, following the semantics rules. I hope this suggestions helps you.😁👍🏾
Marked as helpful0@CarlosgnxPosted over 2 years ago@AdanSaavedra yeah i forget the hover states, i was unsure about put h1 instead h2 tag, thankyou!
0
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