Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NFT Preview Page with CSS and HTML

@akarinaa

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Anosha Ahmed 9,300

@anoshaahmed

Posted

To get rid of the accessibility/HTML issues shown in your Report:

  • wrap everything in your body in <main> OR use semantic tags OR give role="" to the direct children of your <body> ... Click here to read more
  • have at least one <h1> in your code
  • <section> and <article> usually need a heading; so if you don't need a heading in it, use some other element such as <div>

Good job! :)

Marked as helpful

1

Aakash Verma 6,690

@skyv26

Posted

Hi!

To solve accessibility issues:

1.start your headings with <h1> and move up by one level each time

  1. Change your code

FROM

<div class="attribution">
    Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>.
    Coded by <a href="https://www.frontendmentor.io/profile/akarinaa">Karina Angelim</a>.
  </div>

TO

<footer class="attribution">
    Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank" aria-label="challenged by">Frontend Mentor</a>.
    Coded by <a href="https://www.frontendmentor.io/profile/akarinaa" aria-label="developer name">Karina Angelim</a>.
  </footer>

Happy New Year ;)

Marked as helpful

1

@akarinaa

Posted

@skyv26 thanks for the help! I'll implement these changes :)

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord