Skip to content
Submitted almost 3 years ago

[HTML/CSS] NFT Preview Card Component

Guek Ling•60
@guekling
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

This is my first challenge on Frontend Mentor. I'm still trying to figure out good coding practices (i.e. how to best structure my HTML/CSS; I believe my styles.css is pretty messy, but I don't know how to go about tidying it) and understand flexbox. I still have many areas for improvement, so any feedback is much appreciated.

Thank you!

Community feedback

  • Aakash Verma•6,710
    @skyv26Posted almost 3 years ago

    Hi! Guek, just as I saw you made it really good and it is responsive too, I didn't find any error. But as I can see in your report there are 4 accessibility error and here you can easily resolve your issue.

    Just wrap your .card class div and .attribution class div inside <main></main> tag and your issue will then resolve successfully.

    I hope this will work and useful for your future work in order to get 0 accessibility issue.

    Good Luck

    Marked as helpful
    1
  • Harish Radhakrishnan•40
    @HarishPRKPosted almost 3 years ago

    Your project looks very good!

    1
    Guek Ling•60
    @gueklingPosted almost 3 years ago

    @HarishPRK Thank you! Yours looks great too! :D

    1
    Harish Radhakrishnan•40
    @HarishPRKPosted almost 3 years ago

    @guekling Thank you!

    1

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