Design comparison
Solution retrospective
Found the multiple filtered images a good challenge.
Community feedback
- @rohitd99Posted over 1 year ago
Hi David
Congrats on completing the challenge.
On desktop screen size your card dosen't seem to be centered , I figured the reason for it was body has the property
max-height: 100vh
so the body dosen't cover the entire page also with an added disadvantage of limiting the height. So I would suggestmin-height : 100vh
to make it centered and grow as per size of content.Also I found the image to shrink down in size and not covering the entire section at screen width of around 1013px, while checking for responsiveness.
Other than that , the solution is wonderful and perfectly responsive on mobile screens.
Good luck for other challenges
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