Design comparison
SolutionDesign
Community feedback
- @MelvinAguilarPosted 10 months ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- Wrap the page's whole main content in the
<main>
tag.
- For specificity reasons you should work with classes instead of ids because they are more reusable. You can use ids to work with JavaScript, but you should use classes to style your elements. You can read more about this here ๐.
-
You can use the following styles to center the element effectively using either of these two methods: For Grid:
body { min-height: 100vh; display: grid; place-items: center; /* Additional styles if needed */ }
For Flexbox:
body { min-height: 100vh; display: flex; flex-direction: column; justify-content: center; align-items: center; /* Additional styles if needed */ }
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
Marked as helpful1@Jaheim-DouglasPosted 10 months ago@MelvinAguilar thank you for the feedback I am relatively new at frontend development so I appreciate the feedback on how I can be better.
0 - Wrap the page's whole main content in 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