Looks good! Just some minor points to note
- you've forgotten to close main
- I don't think the div wrapping h1 and p is necessary
- you're not using the font size defined in the style guide (0.9rem is not equivalent to 15 px)
- give the card a little margin or a wrapper a little padding so the component can't touch screen edges. It's touching at top and bottom of the screen on my mobile
1
Bilel• 70
@Bilel-mwihbi
Posted
Hi @grace-snow thank you so much for replying 😄
My question is when should I wrap elements with div ?
thank you for considering my request 😊
0