Submitted about 3 years ago
I used html and css with a bit of flexbox magic and made it responsive
@marvcoolguy
Design comparison
SolutionDesign
Solution retrospective
give me feedback on my code
Community feedback
- @afrusselPosted about 3 years ago
I am checking on mobile view. There are lots of area you need to improve.
- background image is not in right place. Use background image 100vh 50%
- change text color of the button while hover
Marked as helpful2@marvcoolguyPosted about 3 years ago@afrussel thank you I will fix all those errors I forgot to make changes to mobile view
0 - @Ryusuke66Posted about 3 years ago
- the background isn't the same as the design.
- using header image as a background is better than putting it as an element.
- the font family for the design is "Red Hat Display".
Marked as helpful1
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