@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.
More Info:π
https://www.w3.org/WAI/tutorials/images/
- Wrap the "NFT image" and "Equilibrium #3429" 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.
https://www.joshwcomeau.com/css/custom-css-reset/
https://meyerweb.com/eric/tools/css/reset/
http://html5doctor.com/html-5-reset-stylesheet/
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!ππ
@elioflo
Posted
@vcarames Thanks for your feedback, helps me a lot!
I updated my repo adding an anchor tag, alt property, and normalize file (scss file). From now on I will use normalize in the frontendmentor challenges.
Merry Christmas!π
Elio Flores