Skip to content
Submitted over 2 years ago

Responsive NFT preview card component

Doina•220
@Doileo
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

Hi there! After finishing this project, I realized that I had made it more difficult for myself. I made a mistake with Figma's design details. Then, in the HTML, I added redundant divs. Another difficult moment for me was trying to fix the card. I mean, when I checked it in Dev tools, the container was in the moving state for the mobile view. I'd appreciate any feedback to help me improve my code. Thank you very much!

Community feedback

  • Abdul•8,540
    @SamadeenPosted over 2 years ago

    Hey Doina!! Cheers 🥂 on completing this challenge.. . Here is my suggestion You can improve your HTML markup with your naming style.. Check out the BEM naming style . Regardless you did amazing.. Happy coding!!!

    0
    Doina•220
    @DoileoPosted over 2 years ago

    @Samadeen Hye Abdul! Thank you so much for your feedback, and for taking the time to highlight this aspect! I hope you have a productive week!

    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