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 card

@zastar23

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

Akshay 830

@akshaywebster

Posted

Hi, Florin.

Nice work with this project. Keep it up, mate!

To get rid of the Accessibility issues:

  • Make sure you have one h1 tag in your HTML document.

  • Your HTML document should have one main landmark. You can achieve this either by using semantic HTML tags or by declaring your main div with role="main".

It's always a great idea to learn about HTML5 semantic tags to reduce errors like this as well as to make your site more readable by screen readers.

Here's a great video on that: HTML5 Semantics

  • Also, your card is not vertically centered, so please look into that if you have some free time.

Overall, you've done a great job since this is your first project here on FrontEnd Mentor.

Hope my feedback is helpful to you. Have a great day, Florin! :)

Marked as helpful

0

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