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

Interactive social link profile using CSS flexbox

@youngimmortal-p

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 learned how to use the CSS transition property

Community feedback

@SvitlanaSuslenkova

Posted

Remove flex from you main and put it to the body with changes:

body { display: flex; flex-direction: column; justify-content: center; align-items: center; min-height: 100vh; } Try this to align(top-bottom) and justify(left-right) your project to the center. It applies to the parent component(body), don't forget about min-height. You can use grid instead of flex too(with some changes).

Hope you found this comment helpful :)

1
T
Grace 29,310

@grace-snow

Posted

I recommend you follow the flex column approach to centering along with min-height: 100svh on the body (the body is only as tall as it's content by default).

More important than that though, is getting the HTML right on this! This should be a list of links (not buttons - because these would navigate somewhere) and these need to be within an unordered list of list items.

This is a really important skill to master — translating a design into the most appropriate meaningful html elements for the content. Don't skip learning and putting that into practice. It's the biggest single thing you can do to build accessible solutions.

0
MikDra1 6,070

@MikDra1

Posted

I encourage you to use this technique to make the card responsive with ease:

.card {
width: 90%;
max-width: 600px;
}

On the smaller screens card will be 90% of the parent (here body), but as soon as the card will be 600px it will lock with this size.

Also to put the card in the center I advise you to use this code snippet:

.container {
display: grid;
place-items: center;
}

Hope you found this comment helpful 💗💗💗

Good job and keep going 😁😊😉

0

T
Grace 29,310

@grace-snow

Posted

Hi @MikDra1, Great feedback! Just a small suggestion to recommend that max width is in rem instead of px. This will make the layout adjust better for those of us who set a different default text size because it will scale right along with our preference. (It avoids the possibility of huge text in a narrow container).

0
MikDra1 6,070

@MikDra1

Posted

@grace-snow

Yeah I know it is just to show what you can do. You always can change things around 😊

0

@vuthoai02

Posted

I think you should center the main part, nice project.

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