nft-preview-card-component with html and css

Community feedback

• 

  Ediongsenyene Joseph I.•1,675

  @IEdiongPosted almost 3 years ago

  Hello Kelvinho 👋, great job in completing this challenge. Here are some suggestions:

  • the .nft-title should contain a heading tag enclosed by the <a>. eg <a class='nft-title'><h1></h1></a>. This should help fix one of the accessibility issues raised here.

  • you could fix the responsiveness issue by using media queries. If you're not familiar with media queries check out this post on responsive design.

  • Also, look into the accessibility and HTML issues raised here.

  I hope this was helpful to you, Shalom!

  Marked as helpful

  1

  ---

  

  Kelvinho•70

  @DrKelvinPosted almost 3 years ago

  @IEdiong Thanks for the suggestion :)

  0

  

  Ediongsenyene Joseph I.•1,675

  @IEdiongPosted almost 3 years ago

  @DrKelvin You're welcome

  0