Responsive NFT preview card component

Community feedback

• 

  Naveen Gumaste•10,420

  @NaveenGumaste

  Posted over 3 years ago

  Hay ! Good Job Ali

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container. For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc . But use header tag only once in main heading element.

  Keep up the good work!

  10

  Marked as helpful
• 

  Rio Cantre•9,650

  @RioCantre

  Posted over 3 years ago

  Hello there! Awesome work with this one. Viewing your solution, I would suggest the following for you...

  • Adjust the breakpoint of the media query into 620px
  • Adjust the padding into padding: 39% 37% 33% 37%; inside the .eye-overlay rule set
  • Wrap the container with specific tag like main for readability
  • Include description for the alt in img tag and should not be empty

  Overall you did well and Keep it up!

  10

  Marked as helpful