Results Summary

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 11 months ago

  Hello @Programming-School-Pro-Coding!

  Your project looks great!

  I have one suggestion for you to improve it:

  • Since you added this to the body: display: flex; flex-direction: column; align-items: center; justify-content: center;, if you also add height: 100vh;, your card will be placed in the middle of the page and you won't need to add margins to move the card from the borders.

  Took a look at your code and updated it for you, like this:

  body {
      font-family: "Hanken Grotesk", sans-serif;
      margin: 0;
      height: 100vh;
      display: flex; 
      flex-direction: column;
      align-items: center;
      justify-content: center;
  }
  

  I hope it helps!

  Other than that, great job!

  0

  ---

  

  Mohamed Ehab•350

  @Programming-School-Pro-CodingPosted 11 months ago

  @danielmrz-dev Hey!

  Thank you for your feedback.

  Yup, this will center the component.

  Thank you for your suggestion.

  0