Design comparison
Solution retrospective
appreciate your feedback
Community feedback
- @hafizanadliPosted over 3 years ago
Great work!
You already did good on how to centering the component. but you 10% margin in container is too big when in big screen. you can change it to margin:10px or 1rem maybe, it will make the component looks better when in large screen.
Keep up the good work!
Marked as helpful0 - Account deleted
Hola Bharath. Your component does not center correctly on devices smaller than 1440px. Using flexbox helps you to center these elements no matter if the screen is increasing or decreasing. And yes, reviewing your code, you are evidently applying it. But if you apply a
margin
, this will prevent the correct centering, you can use apadding
instead of amargin
to avoid this. Greetings! 😀Marked as helpful0
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