Responsive Results Summary Component

Community feedback

• 

  madosy•180

  @madosyPosted over 1 year ago

  Hey looks great! I noticed that it doesn't fit right on mobile. When you added padding-left: 30px to div.summary-content, it added to the total width, causing it to be 405px and generates a scrollbar and empty space to the right.

  I would change the div.summary-content {box-sizing: border-box;} so that your width won't change even if you add padding, and get rid of padding-right on your .summary-content ul.

  I would also get rid of margin-right on your .list-scores and just add padding-right on your div.summary-content so you don't have to look at different areas for controlling the padding.

  Marked as helpful

  0

  ---

  

  0sereme•450

  @Dev-simeon-techPosted over 1 year ago

  @madosy thanks for the advice really appreciate

  0

  

  Gerald LeCour•170

  @Gerald-LeCourPosted over 1 year ago

  @madosy Hey, Thanks so much! I didnt even notice that, I'm going to do that right now!

  0