Skip to content
OverviewSolutions
4
Comments
3
Sam Mathew
@dayumsam

All comments

  • Sam Mathew•80
    @dayumsamPosted over 2 years ago

    It looks great!

    Try using semantic tags: If you look at your report there are 4 accessibility issues almost all of them are because you lack a semantic tag

    Solution: change your card div from a div to a <main> that should fix it.

    Also can also change the font in your document to the one in the style guide

    Keep it up!

    Marked as helpful
    0
  • Wesley kaihara•60
    @WesleyKaiharaSubmitted over 2 years ago

    How can I resolve the small responsiveness error, or in a real case it wouldn't affect it?

    NFT - Preview Card

    2
    Sam Mathew•80
    @dayumsamPosted over 2 years ago

    Great effort! you should try using semantic tags like <main> <header> <footer> instead of <div> which do not tell the browser about the content.

    In your project for example you can replace the element with class Prices_NFTs to a <main class="Prices_NFTs"> instead of div.

    Keep up the efforts!

    Marked as helpful
    0
  • Sam Mathew•80
    @dayumsamPosted over 2 years ago

    It looks great! try changing the font color on the paragraph from 212, 45%, 89% to 220, 15%, 55%

    0