NFT Card preview

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 11 months ago

  Hello there 👋. Good job on completing the challenge !

  I have some suggestions about your code that might interest you.

  • For elements with a hover effect in the design, consider enclosing them within an a (anchor) tag for better semantics and accessibility

  • Use an empty alt attribute (alt="") for icons to signal their decorative nature to assistive technologies.

  • You've applied align-items: center and justify-content: center to the body element, but you might be missing the crucial display: flex. Without this property, the align-items and justify-content won't take effect. Also, use flex-direction: column to ensure that items are centered vertically in a column.

  • Instead of using pixels in font-size, use relative units like em or rem. The font-size in absolute units like pixels does not scale with the user's browser settings. Resource 📘.

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  Marked as helpful

  0

  ---

  

  Mr Azeez•210

  @Mr-AzeezPosted 11 months ago

  Thanks I'll do just that @MelvinAguilar

  0