Design comparison
SolutionDesign
Solution retrospective
Hi all! Tried solving this challenge as little usage of "margin" as possible. Replaced some "margins" with (flex)gap. All feedbacks are appreciated. Cheers!
Community feedback
- @ramazanimaPosted 11 months ago
Your solution is great, and I liked the idea of using sanitize.css. Some suggestions I would give are:
- In your CSS file, you set the margins individually (margin-left, margin-right). I think it would be better to use the shorthand syntax and provide documentation. The same way goes for padding.
- Always set the box-sizing property to border-box to lay out your elements. This makes dealing with the sizes of elements easier.
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