Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social Proof Section

Dan P. 250

@Danielfww

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

I'm proud that its been 5 months and I'm still coding.

I need to take less time to do projects, I did feel a little burned out but that's no excuse, I just got a little bit lazy.

What challenges did you encounter, and how did you overcome them?

This project took me a bit of time, I struggled with order of things - when I had desktop version done, I didn't really add the div container and had to redo the whole structure. When I added the div, there is extra space on the bottom of the page that I don't know how to remove - any help would be much appreciated.

What specific areas of your project would you like help with?

Div removal, white space on the bottom of the page. Any other suggestions are much appreaciated.

Community feedback

Annalisa 170

@Annalisa11

Posted

Hi Dan, about the whitespace on the bottom of the page... I think it is because the background image on the body takes up only as much space as it needs in terms of height. The image height is not big enough to cover all of the background. To get rid of the white space that comes underneath you could give the body the property background-size: cover;which means, that the image should cover the whole space/div it is in and not only the space which it actually needs. Hope it helps (I'm not 100% sure though) :)

Marked as helpful

1

Dan P. 250

@Danielfww

Posted

@Annalisa11 thank you for that.

When I did the project I didn’t really included div into it and then it was a mess in a mobile version so I re-did it all and included the div. I honestly didnt even considered that including the div may actually just scale properly - I thought it didn’t because before I didnt had the extra space.

Thank you for that suggestion.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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