I used the basics of HTML and CSS

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 9 months ago

  Hello @aclf12!

  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;
  }
  

  I hope it helps!

  Other than that, great job!

  1

  ---

  

  Ana Carolina Lucas Ferreira•60

  @aclf12Posted 9 months ago

  @danielmrz-dev Thank you Daniel!!! I hope to learn more about CSS and HTML. I'm still learning some basics. But your tip is already noted and I will apply it to my modification 😊

  1
• 

  yas-avocad•380

  @yas-avocadPosted 9 months ago

  Hey! Your box shadow is noticeably dark: box-shadow: 8px 8px 15px rgba(0, 0, 0, 0.4);

  Try reducing the horizontal and vertical offsets which are the first two values, so it's closer or nearly underneath the box. And probably reduce the spread as well. :)

  1

  ---

  

  Ana Carolina Lucas Ferreira•60

  @aclf12Posted 9 months ago

  @yas-avocad Thanks! I'll try to fix it here.

  0