Results Summary Component

Community feedback

• 

  Thomas Hertog•825

  @thomashertogPosted over 1 year ago

  Your solution is looking good visually! There are some few improvements to be made though.

  HTML/accessibility

  • you used a <main> element which is good, however you also used <div class="paragraph"> while there is a <p> element exactly for paragraph text
  • the results themselves (the elements with class="record" in your code should be a list

  CSS

  • use min-height instead of height to ensure overflow issues are not caused when content is expanding
  • try working with em/rem instead of px

  Marked as helpful

  0