Design comparison
Solution retrospective
Feel free to suggest whatever you want. π
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! π Here are some suggestions to help improve your code:
- Unless your company requires you to use a library , I strongly suggest you stay away from using libraries until you fully grasp the fundamentals of HTML, CSS and JS. By using a library, you are robbing yourself from actually learning how to code.
- 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:π
- 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.
More Info:π
https://www.w3.org/WAI/tutorials/images/
- Your βNFT imageβ needs an Alt Tag. It should describe what the image is; Assume youβre describing the image to someone.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!ππ
Marked as helpful0@gamcode98Posted almost 2 years ago@vcarames Hi! I really appreciate your suggestion. I've changed some things in the project, thank you. In my opinion I think that you must know how works CSS (at least the base) to use in this case TailwindCSS. I struggle when I must give a class name in styles π .
0@VCaramesPosted almost 2 years ago@gamcode98
Glad I could help!βοΈ
The links I provided should help you out for future uses.
Keep it up!
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord