Social Links Profile

Community feedback

• 

  Adriano•38,270

  @AdrianoEscarabotePosted about 1 month ago

  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

  @QLopes22Posted about 1 month ago

  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