@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
- It is best practice ✅ to use,
classes
for styling purposes, while usingids
solely for JavaScript.
- Your
CSS Reset
is being underutilized. 😢 To fully maximize 💯 it, you will want to add more to it. Here are some examples that you can freely use 😁: Josh Comeau Reset Eric Meyer Reset
- To properly center ✅ your content to your page, you will want to add the following to your
body
(this method uses CSS Grid):
body {
min-height: 100vh;
display: grid;
place-content: center;
}
More Info: 📚
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 🎆🎊🪅
Marked as helpful
@stefan123283
Posted
@vcarames Thanks for the feedback. I haven't learned yet about CSS Grid Layout, but should I remove the flexbox method to center the product-card-container if I used the CSS Grid method you gave me to center the product-card-container ?