P
Jeremy Helsel• 670
@JIH7
Posted
Looks great! One suggestion, and this is something I really need to improve with as well, is to use more semantic HTML. On such a small project there's not a ton of room for this, but the <div> with the class of "card" could totally be an <article> instead. I was going to also suggest making the <h2> an <h1>, but seeing as this is designed to be a small component of a larger website, having an <h2> is a great call actually.
Marked as helpful
0
Dean• 190
@DeanMustafa
Posted
You are so right. This is actually what I was thinking. @JIH7
1
