Submitted about 2 years ago
responsive page using scss/sass , javascript , html
@manishdevelops
Design comparison
SolutionDesign
Solution retrospective
any feedback will be welcomed.
Community feedback
- @romila2003Posted about 2 years ago
Hi Manish,
Congratulations 🎉 for completing this challenge, it was a great attempt however there are some issues regarding your HTML and CSS.
- You should give the
body
tag, a background-color instead of thecontainer
as the background color would only be applicable within the container and not the whole body therefore, you will see white space around. - You should remove the
width: min(100%, 375px);
as this affects the layout and causes white space around the container, when the screen size is above 375px.
Overall, great attempt and wish you the best for your future projects so keep coding 👍.
Marked as helpful0@manishdevelopsPosted about 2 years ago@romila2003 okay romilo thanks for your feedback..
1 - You should give the
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