Design comparison
SolutionDesign
Solution retrospective
I didn't understand where the responsiveness fits
Community feedback
- @IEdiongPosted almost 3 years ago
Hello Kelvinho 👋, great job in completing this challenge. Here are some suggestions:
-
the
.nft-title
should contain a heading tag enclosed by the<a>
. eg<a class='nft-title'><h1></h1></a>
. This should help fix one of the accessibility issues raised here. -
you could fix the responsiveness issue by using media queries. If you're not familiar with media queries check out this post on responsive design.
-
Also, look into the accessibility and HTML issues raised here.
I hope this was helpful to you, Shalom!
Marked as helpful1 -
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