Newsletter sign-up form with success message using Flexbox, Grid

Community feedback

• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted over 1 year ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

  ---

  CSS 🎨:

  • The input field of mail want to add style rules after adding the all: unset rule
  • This rule ensure to remove all the default styling of the input element so that in every devices your input element will looks as per you designed
  • So don't forget to add all: unset style rule on top other styles, an example below

  .input-email {
  all: unset;
  --- Then your styles ---
  }
  

  • And another tip; use outline instead of border. Because border will consume extra space while outline doesn't

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  Marked as helpful

  0
• 

  _nehal💎•6,710

  @NehalSahu8055Posted over 1 year ago

  Hello Coder 👋.

  Congratulations on successfully completing the challenge! 🎉

  • Instead of using these attributes use them in the body and replace place-content with place-items

  main {
  background-color: var(--neutral-dark-slate-grey);
  background-size: cover;
  display: grid;
  place-content: center;
  }
  

  like this

  body {
  background-color: var(--neutral-dark-slate-grey);
  background-size: cover;
  display: grid;
  place-items: center;
  }
  

  • Do this and see the effect.

  I hope you find this helpful. Above all, the solution you submitted is great!

  Happy coding😄

  0

  ---

  

  Affan Shaikh•440

  @Terminal239Posted over 1 year ago

  @NehalSahu8055 Thank you for the advice, but using the place-items property places the container at the start and not in the middle which is the requirement for the design.

  0