Skip to content
Submitted 3 months ago

nft-preview-card-component

Vansh Raheja•60
@vanshraheja75
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

What are you most proud of, and what would you do differently next time?

This time i used flex box and grid effectively

What challenges did you encounter, and how did you overcome them?

no challenges faced except faced a little difficulty to set the active state of the view image

What specific areas of your project would you like help with?

any suggestions that you may feel like.

Community feedback

  • Boris•2,870
    @mkborisPosted 3 months ago

    Nice work Vanshraheja75 and great initiative using semantic elements, but for a simple card layout, just wrapping the container with a main element is enough, and there's no need for additional header or footer elements within the card. Also, It's best practice linking Google fonts directly in the HTML head section rather than directly in your CSS file as it enables asynchronous downloading, improving page load times. Font-size should be written in rem not px. This article explains it better Why font-size must NEVER be in pixels

    0

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