Skip to content
Submitted about 2 years ago

Usegin css

Walter Contreras•40
@Contrerasdev
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Second Challenge from Nicaragua. My first steps in web development

Community feedback

  • Natasha Pierre-Louis•280
    @natashaplPosted about 2 years ago

    Hi Walter. Nice work! Just a couple of things. Consider using the main tag to wrap your content rather than the div. It helps with screen readers and is semantic. Here's an informative article that talks about it and all the other HTML5 semantic tags:

    HTML5 Semantic Tags: What They Are and How to Use Them!

    Also, consider changing your h2 tag to an h1. For best practice, there should always be an h1 tag in your page. Hope this helps!

    2

Please log in to post a comment

Log in with GitHub

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