Design comparison
SolutionDesign
Community feedback
- @DanielJ1337Posted about 4 years ago
Hey Nathaniel,
good job! Looks pretty neat :) Congrats on finishing your first project!
Maybe you could add some properties in CSS:
- add height property with 100vh in body so you don't need to add a 100px margin-top manually
- also add a max height of 80vh and a margin: auto auto for the class "container" so everything is nicely centered
Happy Coding :)
0@NathanielCW87Posted about 4 years ago@DanielJ1337 Thank you very much for the feedback, I'll go ahead and make the 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