Design comparison
Solution retrospective
Kindly review my project and drop your comments.
Community feedback
- @MargaritaBusyginaCaPosted over 2 years ago
Good job! I noticed, however, that the background image doesn't take the whole screen and ends in the middle of the screen(especially on big desktop screens). I think you should try setting background-image properties like that :
background-size: contain; background-position: 120% 120%; background-repeat: no-repeat;
For the mobile version, your containers are not centred, they stick to the left of the screen. To fix that you can use either use:
1)margin: 0 auto;
2)or flex justify-content:center
Marked as helpful0@MeekyBerryPosted over 2 years ago@MargaritaBusyginaCa Thanks a lot for the feedback, I’ll make the changes you suggested.
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