Design comparison
Community feedback
- @rayaattaPosted 10 months ago
Hello 👋Umutcan Kocamış, Nice solution 👏.
I have some suggestions you might find useful.
1 Since the illustration image is decorative its alt attribute should be left empty. "Images with no semantic meaning—such as those which are solely decorative—or of limited informational value, should have their alt attributes set to the empty string (""). " - Mdn webdocs
You can check out this article.
2 The avatar image should have an alt attribute with the name of the person on the pic.i.e
alt="Greg Hooper"
3 In your html I noticed
<p class="date">Published 21 Dec 2023</p>
When a screen reader is reading the above it will pronounce
21 Dec 2023
as it is. This should be wrapped in<time datetime="2023-12-21">21 Dec 2023</time>
This is machine readable therefore it is more accessible. To find out more about the time tag check out this article📑.
I hope this helps 🙃
Your solution looks great 😉
Happy coding ✌️
Marked as helpful0@umutcankocamisPosted 10 months ago@rayaatta Thank you so much for your help. I will use what you said for my next challenges :)
1
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