Skip to content
Submitted almost 3 years ago

NFT preview card component using SASS

Nelson•60
@nelsonfleig
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

How would you rate my use of Html tags and naming of css classes?

Do you think my SCSS files are structured properly?

Any general tips?

Community feedback

  • Martin Eliáš•890
    @martinelias1312Posted almost 3 years ago

    Hello @Kokopelli84, i have some suggestions. Accessibility issues:

    • main content should be wrapped in <main> element
    • there should be <h1> heading, article here
    Marked as helpful
    1
    Nelson•60
    @nelsonfleigPosted almost 3 years ago

    @smradupan Thanks for the feedback. Will work on my semantic html!

    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