NFT preview card component

Community feedback

• 

  Amal Karim ✅•1,290

  @amalkarimPosted about 2 years ago

  Hi David 👋,

  I think your solution is a great one. It also looks quite nice on mobile screen, to be honest. I don't see big problems in it. About kinda lost track, well, we do experience it sometimes. The important thing is you finished and submitted the solution.

  However, allow me to give some tiny suggestions (about the details) to your solution.

  • Add some paddings to the body to give a nice white space around the content, especially when the user's browser height is less than the content height. Also, adding margin-bottom to .container would be nice.
  • In .mainImage, instead of background-size; auto 100%;, you could do this instead to make its size not too big but still centered:

  background-size: auto;
  background-repeat: no-repeat;
  background-position: center center;
  

  • Adding align-items: center; to .ethColor and .timeColor will prevent those two icons being squeezed in a small screen size.

  If you still have problems, feel free to reply to this comment.

  Hope this helps. Happy coding!

  Marked as helpful

  1

  ---

  

  David Iván Cabello•110

  @davidicabelloPosted about 2 years ago

  @amalkarim Thank you very much foe the tips! I will implement them as soon as i can !

  0