Newsletter sign-up form with success message solution

Solution retrospective
any suggestion to improve my code is welcome :D
Please log in to post a comment
Log in with GitHubCommunity feedback
- @clinto-bean
Hey Jordan, great work. The only thing I can recommend is to avoid using the min-height: 100vh on the body, because when viewing this on my laptop (1920x1080 @ 13"), it is vertically overflowing causing scrolling that isn't necessary.
You can either attempt to size the main body by adding padding to the container and its children, or you can use a unit like dvw rather than vw, as dvw is a dynamic viewport unit that takes into account things like the UI elements of a user's browser.
Join 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