@PhoenixDev22
Posted
Hello @Abhirup-Sarkar ,
I have some suggestions regarding your solution:
To tackle the accessibility issues:
-
You can use landmark
main
to wrap the body content (which will be the NFT card )HTML5 landmark elements are used to improve navigation . -
Anything with a hover style in design means it's interactive. you need to add an interactive element
<a>
around the image,Equilibrium #3429 and Jules Wyvern
. add hover effect on them . -
The avatar's alt shouldn't be
image of creator
it's meaningless , you can useJules Wyvern
for it. -
the link should be wrapping the main image and either have
Sr-only
text, anaria-label
oralt
text that says where that link takes you. -
No need for this
<hr>
, you can simply useborder-top
for the avatar's partclass="container__creator
-
You should use
em
andrem
units .Bothem
andrem
are flexible, Usingpx
won't allow the user to control the font size based on their needs. It's recommended to use rem for font-size.
Overall, your solution is good, Hopefully this feedback helps.
Marked as helpful
@Abhirup-Sarkar
Posted
@PhoenixDev22 Thank you for your helpful advice :)