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 Links Profile

ks9927β€’ 30

@ks9927

Desktop design screenshot for the Social links profile 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 am proud of the fact that this project took way less time to do then the previous ones. I was worried that it would be difficult without the figma file but it actually felt more easy.

Community feedback

Adrianoβ€’ 36,810

@AdrianoEscarabote

Posted

Hi ks9927, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

Using Flexbox or Grid on the body to center elements ensures a more responsive and adaptive layout, fitting different screen sizes seamlessly. It avoids manual calculations and constant adjustments needed with margin, padding, or absolute positioning. These techniques provide more consistent alignment and simplify the code.

flexbox:

body {
    display: flex;
    justify-content: center;
    align-items:  center;
    min-height: 100vh;
}

grid:

body {
    display: grid;
    place-content: center;
    min-height: 100vh;
}

The rest is excellent.

I hope you find it useful. πŸ‘

0
P
Quentin Lopesβ€’ 50

@QLopes22

Posted

Overall, everything looks accurate visually. The only thing off is centering the main content. For example, I used flexbox on the entire body element to center the main div.

Great work!

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