Ediongsenyene Joseph I.• 1,675
@IEdiong
Posted
Hello Kelvinho 👋, great job in completing this challenge. Here are some suggestions:
-
the
.nft-titleshould 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
1
Kelvinho• 70
@DrKelvin
Posted
@IEdiong Thanks for the suggestion :)
0
Ediongsenyene Joseph I.• 1,675
@IEdiong
Posted
@DrKelvin You're welcome
0
