responsive grid

Community feedback

• 

  Jay Khatri•1,270

  @khatri2002Posted 6 days ago

  Hi! The developed solution looks good! However, there's something that can be improved.

  The Mortgage Amount input field has a min="1000" attribute, meaning values less than 1000 shouldn't be allowed. But when a value below 1000 is entered, the form still gets submitted, the results are displayed, and the default HTML error message Value must be greater than or equal to 1000. is also shown. This might confuse the user.

  To fix this, consider adding a custom validation to prevent the result from being displayed for values below 1000, or remove this validation altogether.

  A similar issue exists for the other two input fields as well.

  Additionally, it's better not to rely on default HTML error messages when you're already handling errors using your own JavaScript. Having multiple types of error messages with different appearances can disrupt the user experience.

  The GitHub repository URL provided seems to be incorrect. Please update it!.

  Great work so far! Happy Coding! 🚀

  Marked as helpful

  0
• 

  Marzia Jalili•950

  @MarziaJaliliPosted 9 days ago

  Yoooo!!!!!!

  Well done, man🙌.

  This is how you can centerize the contanier🤔:

  • Assuming that the container is the only child element of the body element, you can apply these styles to the body selector to get the job done😎:

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

  Hope it was SOMETHING, man😁😁😁.

  Marked as helpful

  0