Design comparison
SolutionDesign
Solution retrospective
Hi, Another project completed another deploy . i am so happy i am learning every day ! Thanks this site exist. Just leave a like and you can add any positive or negative criticism that may help. Thansk
Community feedback
- @ratul0407Posted about 1 year ago
@abie237 congrats on completing this challenge🎉🎉
You've done a pretty good job indeed but I think there are some things that you should consider changing
- Don't set
height
andwidth
explicitly like you've set in yourcontainer
instead you can set something like amax-width:37.5rem
//600px or anything you feel better goes with the design. By doing this you are depending on your content of thecontainer
because theheight
andwidth
will be set according to the children's and you can space things using margin and padding as you would like. This will help you to deal with the responsiveness of the page.
I hope it was helpful for you👍👍
Happy coding my friend🙂🙂
Marked as helpful0@abie237Posted about 1 year ago@ratul0407 thank you . I'm editing it as soon as possible and also applying it to my new projects.👍
0@ratul0407Posted about 1 year ago@abie237 best of luck for your future projects👍
1 - Don't set
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