Design comparison
SolutionDesign
Solution retrospective
New submission!
Built the mobile responsive site first and then did the Desktop. made a huge difference while developing the site!
Community feedback
- @Ambe-Mbong-NwiPosted almost 2 years ago
Your design is lovely, but the anchor tag doesn't use alt attributes as indicated by the validation report. Below is a list of attributes the anchor tag uses:
Global attributes -href -target -download -rel -hreflang -type
1 - @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- Along with the blank
alt tag
, you also want to include thearia-hidden=“true”
to your “car images/icons” to fully remove them from assistive technology.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- The headings in your component are being used incorrectly❌ . Since the
h1
heading can only be used once, it is always given to the heading with the highest level of importance. This component has three headings of equal importance, so the best option would be to use anh2
heading ✅ since it is reusable and it will give each heading the same level of importance.
- For improved accessibility 📈 for your content, it is best practice to use
em
formedia-queries
. Using these unit gives users the ability to scale elements up and down, relative to a set value.
- It is best practice to use,
classes
for styling purposes, while usingids
solely for JavaScript.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
0 - Along with the blank
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