LucasNgTg• 190
@LucasNgTg
Submitted
What specific areas of your project would you like help with?
This is my first time making a form validation. Any feedback is welcome.
Yohannes-GitHub
@Yohannes-GitHub
All comments
- Yohannes-GitHub• 90
@Yohannes-GitHub
Posted
- Your solution's desktop mode meets that of the design almost 1 to 1 so kudos to you on that one.
- Your mobile mode is also really good but the one between the two could be improved. Instead of just decreasing the font from the desktop one, you could decrease the height and width too so that it looks good.
- On the JS part, your form displays an error even while typing the input. Instead of doing that you should validate the input only when the submit button is clicked in my opinion. Also the regex for validating the email is wrong I think in that it allows for someone to submit an input like email@co. The regex that I found covers even this case. It goes like this.
const regex = /^([a-zA-Z0-9._%-]+@[a-zA-Z0-9.-]+\.[a-zA-Z]{2,})$/;- Also, you already have put an event listener on your form that listens to submit so you could put the rest of the function calls in here instead of also putting an event listener on the button and adding the functionality there.
0 - sankiss55• 200
@sankiss55
Submitted
What are you most proud of, and what would you do differently next time?
de nada
What challenges did you encounter, and how did you overcome them?
el diseño con css ya que ea una pagina responsiva y no pude hacerla
What specific areas of your project would you like help with?
en el css y html
Yohannes-GitHub• 90@Yohannes-GitHub
Posted
If I recommend a few tips I would say
- use max-width properties to set the width of the container.
- use margins and gaps(in the case of flexboxes and grids) to space out the elements
1
