All comments

• 

  ishanlearnsprog•120

  @ishanlearnsprogSubmitted over 1 year ago

  

  NFT Preview Card Component

  1

  40
  

  Kristine•310

  @sn-tinPosted over 1 year ago

  Congratulations on successfully finishing this challenge 🎉

  I just have a few recommendations to help you improve your solution.

  First, I noticed that the icons for clock and ethereum has been stretched. This is because the align-self default value is stretch. There are two ways to fix this:

  Solution 1:

  .imgp img {
    align-self: center;
  }
  

  Solution 2

  .imgp {
      display: flex;
      gap: 5px;
      align-items: center;
  }
  

  Then, when hovering over the image, I think it would be best to have the cursor: pointer.

  Nevertheless, the solution you submitted looks great! 😊

  Marked as helpful

  1
• 

  elvis yanez•10

  @eljobnmSubmitted over 3 years ago

  Which is the best way to set the svg's as background? Was really struggling for me D':|

  

  Huddle landing page with single introductory section solution

  1

  10
  

  Kristine•310

  @sn-tinPosted over 3 years ago

  I added it in the body part of css by using: background-image: url(../images/bg-desktop.svg); background-repeat: no-repeat; background-size: cover;}

  I did the same for the mobile size but changed the image to: background-image: url(../images/bg-mobile.svg)

  I also changed the size to: background-size: contain

  and then I added background-position: center top;

  I hope this helps.

  0