Design comparison
Community feedback
- @rayaattaPosted 10 months ago
Hello πlayan-khalil, congratulations on completing this challenge π
I have some suggestions you might find useful.
1 Try to make your html more Semantic by wrapping the main page content inside a
<main>
tag . Replace<div class="container">
With<main class="container">
. This changes nothing visually but Using it makes all the difference. Using semantic markup improvesSEO
And user experience (accessibility) for people using assistive technology such as screen readers. You can find out more about the main tag in this article π°.
2 Since
div
s carry no semantic value replace<div class="preview">
with<section class="preview">
This also improves readability of your code, SEO and accessibility.I hope this helps π
Your solution is amazing π
Happy coding βοΈ
Marked as helpful0@Layan-khalilPosted 10 months agoDon't worry , next time I'll take your advice into consideration thank you @rayaatta
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