Design comparison
SolutionDesign
Community feedback
- @anoshaahmedPosted almost 3 years ago
In the future, to avoid accessibility issues, wrap the direct children of your
<body>
in sematic tags. In your case, that would be.nft-card-component
. You could either use<main>
instead of that, or you could just addrole="main"
to it.Also the blue overlay shows, when the card is hovered, that's supposed to happen when you hover over the image. i'm not sure if you did that on purpose, but either way it's a small thing so besides that, you did awesome so good job!
Marked as helpful1@abdulrsfPosted almost 3 years ago@anoshaahmed
Thank you for the feedback.
Regarding the blue overlay, I did add it on purpose.
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