@rayaatta
Posted
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 improves
SEO
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 helpful
@Layan-khalil
Posted
Don't worry , next time I'll take your advice into consideration thank you @rayaatta