Skip to content
Submitted almost 2 years ago

NFT preview card component

Thompson Urhiofe•30
@februaryfourth
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

I enjoyed this project but sometimes i felt i was using the wrong methods for positioning elements. Would love to get some feedback on how you would have positioned the elements differently.

Thanks in advance!

Community feedback

  • Callie•170
    @Callietron300Posted almost 2 years ago

    Hey, I had a look at your code after seeing your comment, I think you've used flex how I would! I'm only about 6months in but I can't work out anything I would change flex wise, so I think you have it right! I also wanted to compliment your labelling of classes all super clear. I think if you had to be picky the only thing I noticed was the padding at the bottom of your text container which added extra padding to the bottom so solve this you could just use padding-top.

    0
    Thompson Urhiofe•30
    @februaryfourthPosted almost 2 years ago

    @Callietron300 Hi Callie! Thank you very much for your feedback. that really boosted my confidence! I took your advice on changing the text-container to just a top padding and it made a difference in the spacing! Thanks again, T

    1
    Callie•170
    @Callietron300Posted almost 2 years ago

    @februaryfourth you're welcome :)

    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