Design comparison
SolutionDesign
Community feedback
- @HeshamAbdelalemPosted over 1 year ago
Nice work bro, you made it
-
At first sight, I noticed you omitted the
alt
attribute and didn't give it any value, I suggest always adding alternative text to your images -
padding between your elements also will make it fancy, modern, shiny and the most important Easy to Read :)
-
<div class="container">
the div tag could be replaced with themain
tag to make it semantic and to improve the accessibility rules
I hope you find this helpful ;), Great Work
Happy coding
0 -
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