Newsletter sign-up form with success message

Community feedback

• 

  Adriano•42,870

  @AdrianoEscarabote

  Posted 9 months ago

  Hi Michael Balogun, how’s everything? I think your project turned out great! However, I have some feedback that I think might be useful:

  To center the form on the screen, you can use display: flex; to do this:

  .big-container {
      position: relative;
      justify-content: center;
      align-items: center;
      min-height: 100vh;
      /* margin-top: 10vh; */
      display: flex;
      /* z-index: -1; */
      /* margin-bottom: 50px; */
  }
  

  Avoid using margin or padding to center something, always prefer to use flexbox or grid for these situations.

  hahahh I saw that you mentioned that you were having problems with electricity, I was also having the same problem.

  The rest is amazing.

  I hope this is helpful. 👍

  13

  Marked as helpful
• 

  Adriano•42,870

  @AdrianoEscarabote

  Posted 9 months ago

  Hi Michael Balogun, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

  To centralize the component, you can change the code to do this:

  .big-container {
      position: relative;
      /* margin-top: 10vh; */
      /* display: none; */
      /* z-index: -1; */
      /* margin-bottom: 50px; */
      display: flex;
      align-items: center;
      justify-content: center;
      min-height: 100vh;
  }
  

  To improve the component's responsiveness, avoid using fixed measurements and try relative ones, for example:

  max-width: 370px;
  width: 100%;
  

  The rest is excellent.

  I hope you find it useful. 👍

  10

  Marked as helpful