Design comparison
SolutionDesign
Solution retrospective
Hello all, learnt Grid from the best teacher ever. Had lots of fun and grasped the concept. Still lots to learn, excited. Open to feedback. Thanks
Community feedback
- @franexmo81Posted about 1 year ago
Nice submission, really accurate.
I've just noticed some misbehaviour in the responsiveness:
- The cards are not vertically centred on the screen.
- The text overflow the cards on screens between 795 and 1250 px width
- The violet card doesn't take the full width on screens narrower than 795 px
I think that most of the above can be fixed by just doing:
- Remove
padding: 70px
for main-wrapper - Replace
height: 100vh
withmin-height: 100vh
for main-wrapper - Remove
grid-template-rows: 300px 300px
for testimonial - Remove
width: 300px
for the violet card under 795 px width - Remove
margin-top: 500px
andmargin-bottom: 500px
for main-wrapper under 795 px width
I hope the above helps you in your way to become a successful developer. Good luck!
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