Melvin Aguilar ๐ง๐ปโ๐ปโข 61,220
@MelvinAguilar
Posted
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 helpful
1
BrovoJDโข 190
@Jaheim-Douglas
Posted
@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