Design comparison
Solution retrospective
Constructive criticism and valuable insights are warmly welcomed.
Community feedback
- @Kamlesh0007Posted over 1 year ago
Congratulations on completing the challenge!🎉. That's a great achievement, and I'm sure you put a lot of effort into it. I really liked the way you approached the challenge and the code you wrote. You demonstrated a good understanding of the concepts and applied them effectively to solve the problem.I have a few suggestions to improve your code further. You need to add some code in body tag to make the container center using grid layout or Flexbox Here's an example code snippet:
body { background-color: hsl(212, 45%, 89%); font-family: "Outfit", sans-serif; font-size: 1.5rem; display: grid; place-content: center; min-height: 100vh; }
1 - @Andigashi1Posted over 1 year ago
Hey, i suggest you to use seperate files to write the HTML & CSS so it looks less complicated. And in order to center the item properly you can use display: flex in the body:
body { display: flex; flex-direction: column; justify-content: center; align-items: center; min-width: 100vh;
1@KayPeterPosted over 1 year ago@Andigashi1 Thanks for the feedback. I'll implement these changes.
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