NFT Preview Card Component

Community feedback

• 

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  Hello, Deepali Panchal, congrats on completing the challenge successfully! I've been checking some other submissions of your as well but they all looked perfectly, respond very well to all screen sizes, so there was nothing i could add at all, but in this one, i've found something i can suggest now :D It looks like when hovering over the image, the overlay's border-radius does not match to the image. Additionally, if you want to enhance it little further, you can add some slight transition effect from opacity: 0 to opacity: 1 when hovering with e.g. transition: opacity .4s easeInOut; for a smoother effect. Keep up the great work.

  Hope this was helpful!

  Marked as helpful

  0
• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 1 year ago

  Hello, Deepali Panchal! One suggestion I have is to leave the alternative text empty for the Ethereum and clock icons. Those are decorative images. Meaning, they don't have useful information. This will tell the screen reader to skip them. As a result, it saves screen reader users time navigating the page.

  Marked as helpful

  0

  ---

  

  Deepali Panchal•310

  @Deepali25-KPosted over 1 year ago

  @vanzasetia Thank you for the suggestion! I will definitely keep this in mind when working on web development projects in the future.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 1 year ago

  @Deepali25-K

  No problem! 👍

  0