Design comparison
Solution retrospective
I m looking forward for feedback
Community feedback
- @hafizjavaidPosted over 2 years ago
width: 60%; ( Because the image is too wide and it's not according to challenge )
Apply the above thing to the section3 image.
One more thing triy to give classes names according to section instead of section1, section2.
Marked as helpful0 - @hafizjavaidPosted over 2 years ago
@media screen and (min-width: 768px) header { background: url(images/bg-header-desktop.png) no-repeat; background-size: cover; }
Marked as helpful0@nidaismailPosted over 2 years ago@hafizjavaid i did all of the changes my output is now much better. thanks
0 - @optimusprime202Posted almost 3 years ago
Hey @nidaismail, You are learning a lot.
Marked as helpful0@nidaismailPosted almost 3 years ago@optimusprime202 thanks, its fun being here i , looking forward to continue actively
0 - @GitHub-dev12345Posted almost 3 years ago
in image container used Padding property : padding: 100px;
Marked as helpful0@nidaismailPosted almost 3 years ago@GitHub-dev12345 I m not getting you mate, can you be a bit clear.
0 - @hafizjavaidPosted over 2 years ago
Try to use background-size: cover on desktop version so background image can cover the whole width as currently the rightmost area is not filled completely
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