
Design comparison
Community feedback
- @AdrianoEscarabotePosted 5 months ago
Hey FARHAN AHMAD, how’s it going? I was really impressed with your project’s result, though I have some advice that could be helpful:
images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty alt="" and add aria-hidden="true" attributes to make all the assistive technologies of the Web, as screen reader.
Learn the differences between decorative/meaningless images vs important content.
I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written
Learn More
, he is not confirming a form, or something like, it will be redirected to another page, to Learn More about!to solve this problem do this:
<a href="/">Learn More</a>
Everything else looks great.
Hope this helps! 👍
Marked as helpful0@farahn0104Posted 5 months ago@AdrianoEscarabote Thank you so much to giving that's both of things, and I know for alt attribute what he doing, what you told me about the button is very helpful for me, because I did not know about it, from now on I will pay attention to both the things.
Thank You So Much Again...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