Responsive landing page using SASS & JS

Solution retrospective
Hello Everyone, Would love to get some feedback on this challenge. This was my first challenge that I've done where I also had the Figma files as reference and feel very confident with what I was able to produce. There are a few styling mistakes that I know are still needed (padding, margins) to make it look a bit better but I wanted to submit and get any other feedback I could. Thanks!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Nick331102
Nice work overall. You may want to wrap everything in a main and set a max-width, as I viewed your project in a widescreen monitor and would look better with a max width. You could also add a hover state to the request access button. As you mentioned, the margin and padding need to be fixed for the text. The project does look pretty good on mobile though. The success message for the email validation doesn't seem to work. You could reference my project if you would like.
well done
Marked as helpful
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