@shayanabedi98
Submitted
This project was completed with HTML and native CSS (grid and flexbox).
@tchbell
@shayanabedi98
Submitted
This project was completed with HTML and native CSS (grid and flexbox).
@tchbell
Posted
Looks good!
Try and look through your CSS and see where you can combine selectors so you aren't repeating a lot of the same styles. Something small like doing all your margin and padding resets into a group like so.
/* Reset some default styles */
body, h1, h2, h3, p {
margin: 0;
padding: 0;
}
Marked as helpful
@devhnry
Submitted
Hi there 👋, I’m Henry and this is my solution for this challenge. 🚀
🛠️ Tech Stack:
Any suggestions on how I can improve and reduce unnecessary code are welcome!
Thank you. 😊
@tchbell
Posted
Looks super good! The design is a bit larger and has some more spacing around some of the elements. But overall this is really good.
@JohannCirio
Submitted
@tchbell
Posted
Looks really good! Any chance you can make it so that the component is centered on the page?