All comments

• 

  Daniel G Szabo•20

  @d-g-SzaboSubmitted over 1 year ago

  I improved my knowledge about grids and flex, so I implemented those. But this :hoover in the picture is really bad. Pls help me guys :D

  

  NFT Preview Card Component using HTML and CSS Newbie

  3

  50
  

  João Victor Abreu•80

  @shufitoPosted over 1 year ago

  Hello 👋. Congratulations on successfully completing the challenge! 🎉

  I have other recommendations regarding your code that I believe will be of great interest to you.

  Position of css element:

  Looking at your code I noticed that you put in your div a class with the name 'nft-view' that receives a 'position:absolute', but it doesn't have a relative div as a reference. The modifications I suggest are to add a 'position:relative' to the nft-all class and change the top to zero in the 'nft-view' class, as follows:

  .nft-all {
  position: relative
  }
  .nft-view {
  position: absolute;
  top: 0;
  width: 18.9rem;
  height: 18.9rem;
  border-radius: 3%;
  }
  

  Hope you find this useful 😄 Overall, the solution you submitted is great!

  Marked as helpful

  1