Design comparison
Solution retrospective
This didn't take long however figuring out how to position the images took a while so if anyone knows a better way to do it your feedback would be appreciated.
Community feedback
- @elaineleungPosted about 2 years ago
Hi Shikoto, this looks really well put together, so good job! I don't think I have a lot to suggest; the only thing I noticed is that at the mobile view, the top gets cut off, so you might have to change that height back to
height: 100%
so that everything can be seen, and I might also try removing the background position and changing the background size tobackground-size: contain
. Also, some of the font styles seem lost on the button, so you might want to fix that. By the way, I'm not sure whether this call to action should be a button or just a link, but if this ends up directing the user to a signup or a form, I might just use link.Once again, great work here!
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