Design comparison
SolutionDesign
Solution retrospective
Please do not hesitate to give me a feedback of possible improvements.
Community feedback
- @ChamuMutezvaPosted about 1 year ago
Hi Mika
<img src="img/logo-light.svg" alt="Workit Logo">
, the logo will usually be in a link as that will be a link to the homepage.- the
apply for access
button in this case should as well be a link to another section or page, a button is used for cases like opening a menu or submitting a form. Find out more on the difference use cases of buttons and anchor elements. - decorative images (the patterns) should have an empty alt value, the phone image does not appear to be decorative in my opinion - it should be the one with an alt value with a message.
Marked as helpful1 - @0xabdulkhaliqPosted about 1 year ago
Hello there π. Congratulations on successfully completing the challenge! π
- I have other recommendations regarding your code that I believe will be of great interest to you.
ANCHOR ELEMENT π΄:
- Those social image you have used on footer needs to be an actual link for proper navigation.
- They are not for decoration, So we need to wrap each of the images using
<a>
element witharia-label
attribute which is way more important for social links in an<a>
tag can help provide more context to users with visual impairments who use assistive technologies such as screen readers to access your website.
- When a screen reader encounters an anchor tag with a social link, it may announce the link's text content, such as "Facebook" or "Twitter," by including an
aria-label
attribute that points to a nearby element containing a description of the link's purpose, you can provide more context and clarity to the user.
- By providing this additional information, you can help users with visual impairments to better understand the purpose and value of social links, and encourage them to engage with your content. This can ultimately improve the user experience on your website, and make it more accessible and inclusive for all users.
- Example:
<a href="#" aria-label="Facebook profile of Workit"> <img src="img/icon-facebook.svg" alt="" class="footer__logos-fb"> </a>
- If you have any questions or need further clarification, you can always check out
my submission
for another newbie challenge where i used this method and/or feel free to reach out to me.
.
I hope you find this helpful π Above all, the solution you submitted is great !
Happy coding!
Marked as helpful1@mikatechsPosted about 1 year ago@0xAbdulKhalid, appreciate your feedback. Thank you for the tips.
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