Yemisrach• 1,430
@Yemisrach15
Posted
Hi Alexander, good job! A couple of suggestions,
- You could use semantic HTML for accessibility instead of the
div
s. The cards could bearticle
tags. - The top part with the heading and toggle button should be inside a
header
element outsidemain
.
0