All comments

• 

  kenzo-wegher•60

  @kenzo-wegherSubmitted over 1 year ago

  Any suggestions on how I can improve and/or reduce unnecessary code? Thanks in advance. Grazie mille!

  

  Result summary component - using Sass/scss

  2

  70
  

  Kristian Epa Froditus Santoso•320

  @kristiansntsPosted over 1 year ago

  i don't really know about sass, but instead you use too many <div> in summary list, you can use <ul> and <li> tag ...

  and in css you can use :nth-child(n) for every color and background color that different ...

  keep in mind to not DRY (Don't Repeat Yourself) in your code ... reread your code, and find what you think you can use some case in just one case ...

  i think if you can learn about parent and child and what can affecting in css it will be useful

  Marked as helpful

  0
• 

  Imo Emmanuel•140

  @PearlPerfectSubmitted over 1 year ago

  Not sure of how responsive the site is. would love some corrections.

  

  I made used of the span element to create the reactions

  2

  40
  

  Kristian Epa Froditus Santoso•320

  @kristiansntsPosted over 1 year ago

  instead using so many div in summary list, its easy if u use for summary list with <ul> and <li> ... for ul you can just adjust the margin and width of ul (DRY - Don't Repeat yourself)

  and for li you can set up the padding and for the different color you can use nth-child(n)..

  and avoid using inline styling -> like this <span *style="color:hsl(39, 100%, 56%);"*>Memory</span>

  0