Intro component with sign-up form

Community feedback

• 

  Manoj Kumar Singh•970

  @dev-mksinghPosted 9 months ago

  Hey @SandyAstorga , well first of all, that's good work in your project. I would like to add something that i personally think will be better in future:

  • Right now you are using px for width in different breakpoints(responsiveness) , prefer using relative-length units like rem, em and %

  • Secondly, try using mobile first approach, currently you created the project firstly for the web version then you set the different breakpoints according to the screen sizes(that's what your code shows) , use mobile-first approach you will find it much better and you can do more by writing less code.

  Hope this will be helpful to you. Happy coding.

  Marked as helpful

  0

  ---

  

  SandyAstorga•460

  @SandyAstorgaPosted 9 months ago

  @dev-mksingh Hello! Thank you very much for the observation, I have modified it, I would like to know your opinion again and know what else I can improve. Beforehand thank you very much!

  0

  

  Manoj Kumar Singh•970

  @dev-mksinghPosted 9 months ago

  @SandyAstorga yep now it look much more better and continuous. Yes that's good, well still its not mobile first approach, but lot better than previous one. Also i find something on your code:

  body {
     min-height: 100vh;
  }
  

  Personally speaking, avoid using heights until its very much needed. The reason (that i understood by now) is if height is fixed you have boundaries there and have to set the elements height accordingly within many containers, which we dont want to do. Because we have tools like flex grid to automatically have same height for all the elements within container.

  Hop my point is clear, and it will be useful again. Happy coding. Any help further will be my pleasure. And hey you are doing good, keep going better.

  Marked as helpful

  0