Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NFT preview card component

@februaryfourth

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@Callietron300

Posted

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

@februaryfourth

Posted

@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

@Callietron300

Posted

@februaryfourth you're welcome :)

0

Please log in to post a comment

Log in with GitHub
Discord logo

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