Submitted almost 2 years ago
Responsive card component developed with SASS and HTML
@yorchAdalbert
Design comparison
SolutionDesign
Solution retrospective
I didn't know how to implement the active state into the nft image. Any feedback is really welcome.
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- The NFT
alt tag
description needs to be improved upon. You want to describe what the image is; they need to be readable. Assume you’re describing the image to someone.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- Along with the blank
alt tag
, you also want to include thearia-hidden=“true”
to your “icons” to fully remove them from assistive technology.
- The profile image is not decorative. The alt tags should not be blank. It should state the following; “Headshot of -person’s full name-“
- 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:📚
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
Marked as helpful1 - The NFT
- @atif-devPosted almost 2 years ago
Hi Jorge, you can learn from here:
https://www.w3schools.com/howto/howto_css_image_overlay.asp
Hope you will find this Feedback Helpful😇
Marked as helpful1
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