Frontend Mentor | social-links-profile-main coding challenge solution

Community feedback

@danielmrz-dev

Posted 9 months ago

Hello @GASST0N!

Your project looks great!

I have one suggestion for you to improve it even more:

Using margin is not the best option to center an element. Here's a very efficient (and better) way to place an element in the middle of the page both vertically and horizontally:

📌 Apply this to the body (in order to work properly, don't use position or margins):

body {
    min-height: 100vh;
    display: flex;  /* it works with grid too  */
    justify-content: center;
    align-items: center;
}

Using Tailwind, that would be <body class="min-h-screen flex/grid justify-center items-center">.

I hope it helps!

Other than that, great job!

social-links-profile-main