nft-preview-card-component

Community feedback

• 

  Nabil Wasir•680

  @NabilWasirPosted almost 2 years ago

  Great work, well done

  You can get rid of accessibility issues by replacing div/span of the main container with <main></main>

  Marked as helpful

  0

  ---

  

  Moyosore Sulola•30

  @MoyosorePraisePosted almost 2 years ago

  @NabilWasir Thank you

  It worked.

  0

  

  Nabil Wasir•680

  @NabilWasirPosted almost 2 years ago

  @MoyosorePraise Happy to help

  0
• 

  Adriano•38,270

  @AdrianoEscarabotePosted almost 2 years ago

  Hello Moyosore Sulola, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:

  Consider using rem for font size .If your web content font sizes are set in absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. Relative units “stretch” according to the screen size and/or user’s preferred font size, and work on a large range of devices.

  if you want to continue coding with px, you can download a very useful extension in vscode, it converts px to rem! link -> px to rem

  <hr> prefer to use border-bottom to make this line!

  The remainder is excellent.

  I hope it's useful. 👍

  0