Design comparison
SolutionDesign
Solution retrospective
All feedback is greatly appreciated - thanks!
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- The logo’s
alt tag
description ⚠️ should only state the company’s name, nothing more.
- Your
email input
⚠️ needs to have a visibly hiddenlabel
attached to it for improved accessibility.
- Along with the blank
alt tag
, you also want to include thearia-hidden=“true”
to your "Illustrations/icons" to fully remove them from assistive technology.
More Info:📚
[MDN Aria-Hidden] (https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-hidden#description)
- ⚠️ The logo and company info inside the
footer
need to be wrapped inside anaddress
element.
More Info:📚
- ⚠️ The company directory needs to be a single list, not two. You will use the CSS property
column
to style it.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 🎆🎊🪅
Marked as helpful1 - The logo’s
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