Perfectly balanced :D

Mark Wasfy• 280

@MarkWasfy00

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

NFT preview card component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

please leave a reviews ,, thanks in advance :)

Community feedback

Alex• 660

@al3xback

Posted over 2 years ago

HI Mark,

My feedback are as follows:

for class nft-photo, using flex properties is unnecessary cos we dont have content inside.
for class ETH and time, we dont need to use flex-direction: row as its default value for flex.
it's unnecessary to add color property on class line, it's alr handled by border.

Marked as helpful

Mark Wasfy• 280

@MarkWasfy00

Posted over 2 years ago

@al3xback really appreciate your support and your notes , i just sometimes use properties but when i make a changes i just leave the old ones , again tysm i got my code updated in github :D

Please log in to post a comment

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