NFT preview card component

Community feedback

• 

  Anosha Ahmed•9,300

  @anoshaahmed

  Posted over 3 years ago

  hey good job on this challenge! you should have at least one <h1> in your webpage

  here is a list i made of accessibility issues & best practices

  hope this helps :))

  11

  Marked as helpful
• 

  Naveen Gumaste•10,420

  @NaveenGumaste

  Posted over 3 years ago

  Hay ! Good Job you made it look nearly perfect to the preview

  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.

  -> Don't use "p tag" inside the "span tag" instead use the "div tag" [span is a inline element ]

  Keep up the good work!

  10

  Marked as helpful
• 

  Rio Cantre•9,650

  @RioCantre

  Posted over 3 years ago

  Hello there! Nice work with this challenge. Viewing your solution, you did well on implementing the hover state of the Hero image. I think you should considered the following as well...

  • Add hover state for the font in the header with color: hsl(178, 100%, 50%); and cursor: pointer;
  • Remove the inline style inside the span and create a new rule set of .card-footer span, add color:white and hover state with color: hsl(178, 100%, 50%); and cursor: pointer;
  • Import the attribution style in the CSS folder and remove style tag
  • Add description to alt in the img, don't leave it blank

  Hope this helps and Keep up the good work!

  01

  Marked as helpful