Skip to content
Submitted 3 months ago

NFT Preview Card Component w/SCSS

P
Mukarram Haq•440
@MukarramHaq
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?

N/A

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

N/A

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

I created this project using SASS and I would really love some feedback on how I wrote the code with SASS. Any feedback to improve the code will be helpful.

Community feedback

  • calvinvin•790
    @calvinvinPosted 3 months ago

    Looks good and I don't know SASS... .

    If I have to comment:

    1. shouldn't skip heading order, i.e. don't jump to <h3> without <h1> and <h2> used.
    2. the margin-block-start of the card on mobile version is not modified.
    0
    P
    Mukarram Haq•440
    @MukarramHaqPosted 3 months ago

    @calvinvin Thank you for the feedback. I didn't really got into making it pixel perfect. Just wanted to try SASS out

    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