FLEX

Solution retrospective
This challenge was given to me with difficulty. Somehow I was able to repeat it, but exactly what to repeat, and not fully to make up. The CS code is terrible, it's even possible that some properties repeat each other. Nevertheless, I will be glad of any feedback
Please log in to post a comment
Log in with GitHubCommunity feedback
- @sulemaan7070
hey @ilvdrskn😄, congratulations on completing the challenge here are a few suggestions to make your site better..
1.Firstly the padding
padding: 2em 5em
on the button looks massive you can reduce that.. or if you are comfortable with pixels just go with pixels..2.On the
.modal-menu
use the property of fixed like this, this should keep the menu on the screen even when scrolling happens...modal-menu { position: fixed; }
3.the
padding-left: 10em;
on themain.main-info
is pushing the box more than design..you can reduce that..4.As for the
.picture
div you can use the property ofbackground-size: contain;
to display full image..Hope that helps, Happy coding💯🔥👍🏻
Marked as helpful
Join 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