Responsive Lending uses @media

Community feedback

• 

  Michael•240

  @mksoofianPosted over 1 year ago

  Hi @nikitaaksjonov,

  This is an amazing first attempt! Very impressive.

  The only main design difference is the layout of your items within you list <li> elements. I think it would be improved by using:

  1. Rather than <span> elements, <p> or <div> would be more suitable here.
  2. There are various ways to do space these but I will give one example here: Group the item-picture and item in one div and give it a class something like "category" and then group the other two value elements in a div and give it a class of "cat-results". You can then add to .summary-container ul li {justify-content: space-between} which move them to the ends of the "reaction container. Then I would adjust the spacing accordingly.

  I hope this helps! Best of luck!

  1
• 

  nikitaaksjonov•70

  @nikitaaksjonovPosted over 1 year ago

  Hi @mksoofian, Thank you so much for your feedback. I will fix what you advised to do and it will be perfect. ;) P.S. If you have a chance, please take a look at my second challenge

  0