@IEdiong
Posted
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 helpful
@DrKelvin
Posted
@IEdiong Thanks for the suggestion :)
@IEdiong
Posted
@DrKelvin You're welcome