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 (FLEX,RELATIVE)-UPDATED

@YOUXIE-DEV

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


All comments are welcome :) Thanks You!

Community feedback

Hassia Issah 50,650

@Hassiai

Posted

Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To

There is no for max-height: 100vh in .CARD, replace the width of .CARD with max-width for more responsive width.

You forgot to add the overlay image and the background-color to img:hover

Hope am helpful HAPPY CODING

Marked as helpful

1

@YOUXIE-DEV

Posted

@Hassiai Thanks for your feedback :)

0
Atif Iqbal 3,200

@atif-dev

Posted

Hi -YOUXIE-DEV, you did will.

1)Learn about image overlay from https://www.w3schools.com/howto/howto_css_image_overlay_title.asp

2)Ttry to center the main container vertically as well. For centering do as:

body {
    min-height: 100vh;
    display: grid;
    place-content: center;
}

or learn centering from here https://moderncss.dev/complete-guide-to-centering-in-css/

3)Read frontendmnetor report generator issues and improve things.

Hope you find this Feedback Helpful

Marked as helpful

1

@YOUXIE-DEV

Posted

@atif-dev Thanks for your Feedback !

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