Thomas Hertog• 715
@thomashertog
Posted
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-heightinstead ofheightto ensure overflow issues are not caused when content is expanding - try working with
em/reminstead ofpx
Marked as helpful
0
