Design comparison
Solution retrospective
The separators are done well, alongside the spacing and margins
What challenges did you encounter, and how did you overcome them?adding padding the list items, I searched for a way and used a wrong solution, but then found the correct way
What specific areas of your project would you like help with?nothing in particular
Community feedback
- Account deleted
Congratulations on your website, I just have few things to point out;
-
Your website is not responsive for smaller device from 400px below, by using rem or percentage instead of px, you don't need @media for now. This would make your website responsive,
-
You shouldn't set height for your container, just set width instead.
-
Your code is too long, that is because you are using ID selector, you should use class instead, it would make it easy to shorten your code. try working on that for now.
Marked as helpful1 -
- Account deleted
Congratulations on your website, I just have few things to point out;
-
Your website is not responsive for smaller device from 400px below, by using rem or percentage instead of px, you don't need @media for now. This would make your website responsive,
-
You shouldn't set height for your container, just set width instead.
-
Your code is too long, that is because you are using ID selector, you should use class instead, it would make it easy to shorten your code. try working on that for now.
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