NFT Preview Card

Community feedback

• 

  バレンタイン 😈•64,170

  @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/

  • The “Icons” serve no other purpose than to be decorative; They add no value. Their alt tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.

  More Info:📚

  https://www.w3.org/WAI/tutorials/images/

  • Wrap the "NFT image" 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:📚

  MDN The Anchor element

  • The attribution should be wrapped in a footer element.

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding!🎄🎁

  Marked as helpful

  0

  ---

  

  Clark•70

  @midnight-clrkPosted almost 2 years ago

  @vcarames Thanks again. 😁

  1

  

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  @midnight-clrk

  Glad I could help!

  Keep it up!

  0
• 

  Chris•180

  @chrizgxPosted almost 2 years ago

  Hey! I really like your design. I also took a look at your code, and I was amazed by how it's possible to edit opacity through HSL colors directly. When I wanted to apply opacity to the color itself (instead of the whole element), I had to to find the equovalent RGBA code. Thanks for that!

  As for feedback, the only thing I can say is you forgot to add border-radius: 10px to the overlay.

  I really like the outcome, great work!

  Marked as helpful

  0

  ---

  

  Clark•70

  @midnight-clrkPosted almost 2 years ago

  @chrizgx Hey, thanks for the feedback. I learned it from Kevin Powell, amazing CSS teacher.

  0