@ApplePieGiraffe
Posted
Hey, Coty B! 👋
Nice job on this challenge! 🙌 It's a rather simple challenge but your solution looks good and so does your code! 👍
The only tiny thing I might suggest is to consider using a single <h1>
tag per page (as that's commonly recommended best practice, at the moment) rather than more than one. In this case, some less-important heading tags (such as <h2>
or <h3>
) might work better for the headings in the card component. 😉
And also, you could decrease the margin above the card component just a bit in the desktop layout so that the card doesn't get pushed too far down and look off-centered on desktop screens with smaller heights. But that's a very small thing, as well!
Keep coding (and happy coding, too)! 😀