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 With CSS Flex

Rian 70

@AdhaRian

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


Hey Guys, This is my solution for this challenge.

I find it difficult making the image hover and unsure if the code is right.

Any feedback for the this code would be appreciated

Community feedback

Maksim 590

@MaxTarasevich

Posted

Hi Rian!

Good job!

Pay attention to the following points:

In order to center the .card block, you made the body flex container as well as the .container block. I think this is a bit redundant, try to remove the flex properties from these blocks, as well as the properties

max-width: 30%; height: auto;

height: auto; is the default property and max-width: 30%; will greatly compress your card at low screen resolutions, and vice versa at high resolutions - greatly stretch it.

It is enough to set the width for the .card block and give it a property margin: auto; the block will be centered. Or, instead of margin, return the flex properties to the body tag.

For block .overlay properties

width: 100% 
 height: 98.5%;

superfluous, just set

   top: 0; 
   left: 0; 
    bottom: 0; 
    right: 0;

or in the shorthand

inset: 0; 

and the block will take the entire width and height. In order for the image to match the parent block, add the property display: block;

I hope my feedback will be useful to you!

1

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