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 master

suhridp 110

@suhridp

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


can you please guide me on making it responsive website ? I have tried but I think it could have been better.

Community feedback

hitmorecode 6,230

@hitmorecode

Posted

Actually when you setup a web page without adding css, flexbox, grid and so on. The page is already good to go for mobile, because every time you place a div or p or h tag, they are going to pile on top of each other. So your html out of the box is already mobile first. It also took me awhile to understand this.

You just need to add style to make it look nice. After that you can start working on a desktop version.

You are using flex-wrap to switch to mobile (it's not wrong), you could have used media query and set a width for mobile and go from rows to column in one click.

0

suhridp 110

@suhridp

Posted

@hitmorecode how can i add the width for the mobile? then everything in the page width will not be in coherence with the design?

0
hitmorecode 6,230

@hitmorecode

Posted

@suhridp so I took a better look at you css, there are somethings that you used but don't need.

body{
display:flex;
flex-direction:column;
/* if you add these two lines the content will be vertically aligned to the middle of the page */
min-height: 100vh;
justify-content: center;
}

header{
height:1rem;
/* width:100vw; */ /* you don't need this */
}

.container{
display: flex;
justify-content: center;
align-items: center;
flex-wrap: wrap;
/* overflow: hidden; */ /* you don't need this */
}

.container {
/* z-index: -1; */ /* you don't need this */
background: url(bg-pattern-top-mobile.svg);
background-repeat: no-repeat;
}


/* if you didn't used flex-wrap, you could have done something it like this */
@media only screen and (max-width: 1020px) {
.container {
max-width: 300px;
}
}

The thing is when switching from desktop to mobile, some times you have to adjust the width of the elements to make it fit a mobile screen.

Marked as helpful

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