Design comparison
Solution retrospective
Any feedback is appreciated!
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hi there, Ayon! 👋
Nice job on this challenge! 👏 Your solution looks good and you've done a nice job in placing the background images in this challenge (something that can be a little tricky to do)! 👍
The only tiny thing I have to suggest is to leave the
alt
text for the image that is the background pattern of the card empty since it is a purely decorative image and doesn't need to be read by screen readers. 😉You could also use an
<article>
tag for the card component itself (since it is a self-contained element) to make your HTML a little more semantic.Keep coding (and happy coding, too)! 😁
Marked as helpful1@ayonnnPosted over 3 years ago@ApplePieGiraffe Thank you so much for your feedback! 😁 I will be sure to adjust my code later in the day. Again, many thanks!
1 - @ezraakankwasaPosted over 3 years ago
You did a good job though you could have center aligned the card in the middle but all in all great job.
Marked as helpful1@ayonnnPosted over 3 years ago@ezraakankwasa Thank you for your feedback! Later, I will adjust the position of the card. 👍
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord