Design comparison
SolutionDesign
Solution retrospective
Is it layed out nicely from perspective of width and responsiveness ?
Community feedback
- @axevldkPosted almost 4 years ago
Hi, @Omar Ahmed ~ I have studied your work, and it's really great ~ 👍
Here are some of my tips.
-
You can use ul instead of div for .advs elements. I think that would be more appropriate for semantic requirement.
-
I think if you use BEM naming convention here, it would become greater.
Hope this will help you even a bit ~ 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