Tip calculator with vanilla js, some css grid and most of flexbox

Community feedback

• 

  Adriano•39,370

  @AdrianoEscarabotePosted about 2 years ago

  Hi Sebastian, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  To center your app in the center of the screen, do this:

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

  • Document should have one main landmark, to solve this, wrap all the content with the main tag, taking into account that this challenge is just an app, so there's no header and footer!

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  0

  ---

  

  Sebastian•510

  @sebix0nusPosted about 2 years ago

  @AdrianoEscarabote In my case it did not help, I would have to correct too many things, I centered the page in a different way, also have a wrapper for all the content. I will try this way for my next projects. Thanks! 👍

  0

  

  Adriano•39,370

  @AdrianoEscarabotePosted about 2 years ago

  @sebix0nus ok...

  0