Social Links

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 6 months ago

  Hello there!

  Congrats on completing the challenge! ✅

  Your project is looking fantastic!

  I'd like to suggest a way to make it even better:

  • Using margin isn't always the most effective method for centering an element.

  Here's a highly efficient approach to position an element at the center of the page both vertically and horizontally:

  📌 Apply this CSS to the body (avoid using position or margins in order to work correctly):

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

  I hope you find this helpful!

  Keep up the excellent work!

  Marked as helpful

  0

  ---

  

  Dan P.•250

  @DanielfwwPosted 6 months ago

  @danielmrz-dev thank you sir - I will try to implement that in my next project.

  1
• 

  Alex•3,130

  @Alex-Archer-IPosted 6 months ago

  HI! Well done, your work quite similar to original design!

  I can recommend you to use one class name for the identical elements - like just .button in your case. It'll help to reduce your css code and make it more readability. Also you can reduce margin-left and margin-right to margin: 0 auto.

  Hope that helps. Keep doing =)

  1
• 

  Erfan•310

  @ERFAN-REVENANTPosted 4 months ago

  Hey Dan P.- great job on finishing this project. If I had to recommend anything here, I would suggest you would use the following to center your div to the middle of the page:

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

  0