Fraser Wattβ’ 1,790
@fraserwat
Posted
Hey Kyvs, good work on this one!
Few changes would make:
- Your heading has the correct use of <h1>, but because its a semantic element, probably doesn't need the class of "heading".
- You should only have one <h1> element per page, so the <h1> elements inside your .numbers divs I would probably structure like <p><span>10k+</span> companies</p>
- Would use padding on the <body> so that you don't have the element riding up to the edge of the screen ~800-1200px widths.
- Try using either margin or flex attributes to move the .attribution element to the bottom of the screen (also think about what you might be able to use instead of a div here)
Happy coding, its looking good!!
Fraser
0
