nft-preview-card-mll-challege-mll

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Nice work with this project. Looking at your solution, I would recommend the following for you...

  • Add border-radius: 10px; in the .container .main-image div and img rule sets
  • Add padding: 1rem 0; in the .container h2 rule set
  • Modify this line <li><img src="images/icon-clock.svg"> 3 days left</li>
  • Include alt with description in the img tags
  • Wrap container with specific tag like main and attribution with footer tag
  • Clean the whitespaces in the code

  Overall you did good in implementing the design. Hope this helps and Keep it up!

  0

  ---

  

  loulou•60

  @louloumwanaPosted almost 3 years ago

  @RioCantre Thank you very much

  1

  

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  @louloumwana No worries, I would appreciate it if you mark it as helpful! Cheers!

  0