Responsive Newsletter Signup with Success Message

Community feedback

• 

  _nehal💎•6,710

  @NehalSahu8055Posted over 1 year ago

  Hello Coder 👋.

  Congratulations on successfully completing the challenge! 🎉

  Few suggestions regarding design.

  ➨To properly center the container.

  • USING FLEXBOX

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

  • USING GRID

  body{
  min-height: 100vh;
  display: grid;
  place-items: center;
  }
  

  ➨ You are using vw almost everywhere vw is not good practice to use everywhere as this will cause very large sizes for larger screen.

  ➨ Use some other responsive units(rem, em, %) from next project. Explore respective use cases on google.

  link.

  ➨ You can use accessibility features like aria, sr-only, and title which are accessible to screen readers.

  I hope you find this helpful.

  Happy coding😄

  0