NFT Preview Card

Community feedback

• 

  Guek Ling•60

  @gueklingPosted almost 3 years ago

  Hi Justin, great work you've done there! Here are some suggestions I have:

  (a) I notice that your site isn't responsive. It looks great on web, but is a little too large on mobile. Perhaps you can look into media queries to turn your site responsive.

  (b) It seems that you're having some difficulties with the hover states for interactive elements. I was stuck on them for a long time too! For the image hover, perhaps you can check out this StackOverflow answer; it helped me a lot when I was doing this challenge.

  (c) To solve some of the issues reported, you can add alternate text to your image. If you find it unnecessary, an empty alternate text would work too. For example: <img src="./images/icon-ethereum.svg" alt="">

  Good luck!

  Marked as helpful

  0

  ---

  

  Justin Jewell•20

  @jjewelesPosted almost 3 years ago

  Thanks for the feedback. All the things you mentioned were just laziness from doing it at 4am lol. I’ll update the solution at some point to fix the responsive and accessibility. 🤙🤙 @guekling

  1