NFT preview card component (FLEX,RELATIVE)-UPDATED

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  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•30

  @YOUXIE-DEVPosted almost 2 years ago

  @Hassiai Thanks for your feedback :)

  0
• 

  Atif Iqbal•3,200

  @atif-devPosted almost 2 years ago

  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•30

  @YOUXIE-DEVPosted almost 2 years ago

  @atif-dev Thanks for your Feedback !

  1