@VCarames
Posted
Hey there! π Here are some suggestions to help improve your code:
- The NFT
alt tag
description needs to be improved upon. This is what your users are purchasing. You want to describe what the image is; Assume youβre describing the image to someone over the phone.
-
- The βIconsβ serve no other purpose than to be decorative; They add no value. Their
alt tag
should left blank and have anaria-hidden=βtrueβ
to hides it from assistive technology.
- The βIconsβ serve no other purpose than to be decorative; They add no value. Their
More Info:π
https://www.w3.org/WAI/tutorials/images/
- Wrap the "NFT image", "Equilibrium #3429" and "Jules Wyvern" in an
anchor tags
. The anchor tag will allow users to click on content and have them directed to another part of your site.
More Info:π
- Your CSS Reset is extremely bare and being underutilized. To fully maximize your CSS reset, you want to add more to it.
Here are few CSS Resets that you can look at and use to create your own or just copy and paste one that is already prebuilt.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! ππ
Marked as helpful
@Guilherme-Goncalves-de-Souza
Posted
Thank you very much for all the feedback, mainly for the "aria-hidden" did not know that this existed.
Thank you very much @vcarames !!!