Design comparison
Solution retrospective
Feel Free To Drop A Feedback <3
Community feedback
- @danielmrz-devPosted 10 months ago
Hello @Ali-AK004!
Your solution looks great!
I have a couple of suggestions for improvement:
- For semantic reasons, replace your
div.container
withmain.container
.
This tag change has no visual impact but they make your HTML code more semantic and improve SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, great job!
Marked as helpful1 - For semantic reasons, replace your
- @MelvinAguilarPosted 10 months ago
Hello there π. Good job on completing the challenge !
I have other small suggestion about your code that might interest you.
- Not all images should have alt text. Car icons are for decoration purposes only, so they can be hidden from screen-readers by leaving its alt attribute empty. You can read more about this here π.
I hope you find it useful! π Above all, the solution you submitted is great!
Happy coding!
Marked as helpful1@Ali-AK004Posted 10 months agoHello @MelvinAguilar Thanks For these info , really appreciate Your Help <3
0 - @rayaattaPosted 10 months ago
Hi Ali-ak004 congratulations completing this challenge π. Here are some tips you might find useful.
1.the main content of your page should be wrapped inside a main tag.so change
<div class="container">
to<main class="container">
2.since divs have no semantic value, change
<div class="box green"> <div class="box dark">
To
<section class="box orange"> <Section class="box green"> <section class="box dark">
3.change
<div class="btn">
to<button class="btn">
this improves navigation for people who use keys instead of the mouse. 4.since the icons are decorative their **alt ** attribute should be empty. I hope this helps with the html. Happy codingMarked as helpful1@Ali-AK004Posted 10 months agoHi @rayaatta Thanks A lot For These Info , Appreciate your Help <3
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