I m looking forward for feedback
@hafizjavaid
Posted
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 helpful
@hafizjavaid
Posted
@media screen and (min-width: 768px) header { background: url(images/bg-header-desktop.png) no-repeat; background-size: cover; }
Marked as helpful
@nidaismail
Posted
@hafizjavaid i did all of the changes my output is now much better. thanks
@optimusprime202
Posted
Hey @nidaismail, You are learning a lot.
Marked as helpful
@nidaismail
Posted
@optimusprime202 thanks, its fun being here i , looking forward to continue actively
@GitHub-dev12345
Posted
in image container used Padding property : padding: 100px;
Marked as helpful
@nidaismail
Posted
@GitHub-dev12345 thanks i will check it out
@nidaismail
Posted
@GitHub-dev12345 I m not getting you mate, can you be a bit clear.
@hafizjavaid
Posted
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
@nidaismail
Posted
@hafizjavaid okay thank you
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
