
Submitted over 2 years ago
Responsive landing page using css grid and flexbox
@Mennatallah-Hisham
Design comparison
SolutionDesign
Solution retrospective
hope I get feedback regarding the structure of the HTML
Community feedback
- @AdrianoEscarabotePosted over 2 years ago
Hi Mennatallah-Hisham, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
- The links must have an aria-label or sr-only text that tells where the link navigates the user. For example: Visit our Facebook. For images, you should set aria-hidden=” true” to be ignored by screen readers and to avoid redundancy and repetition.
- images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty
alt=""
and addaria-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.
The rest is great!
I hope it helps... 👍
Marked as helpful0@Mennatallah-HishamPosted over 2 years ago@AdrianoEscarabote thanks for your feedback, it was really helpful
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