@ApplePieGiraffe
Posted
Hi, Sulaiman! 👋
Very sorry for my late reply! 😅 Been a bit caught up with school and things recently! 🏫
Great work on this challenge! 👍 Your solution looks pretty good! 😀
A couple of small things I'd like to suggest are,
- Writing some more descriptive text for the
alt
text of the main image of the card component. "The Preview Card Component Image" doesn't really say anything about what the actual content of the image is to people using screen readers. 😉 - Wrapping the heading of the card component in a heading tag and perhaps using an
article
tag for the entire component (since it is a self-contained element that makes sense without additional context from a parent component). This'll just bump up the semantics of your HTML a little. 😄
Again, these are just small points. Overall, you've done well on this challenge! 👏
Hope you find this helpful. 😊
Keep coding (and happy coding, too)! 😁
Marked as helpful
@Abu-Sman
Posted
@ApplePieGiraffe Thank you so much!