Skip to content
Submitted over 3 years ago

Solution made with HTML + CSS + custom Vanilla JS form validation

Etienne L.•95
@etnlmy
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Intro component with sign-up form
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hi, I would appreciate feedback on the CSS and on the custom form validation. Thanks :)

Community feedback

  • Phil Jacks•125
    @philjacksPosted over 3 years ago

    Hi friend!

    CSS looks great & all the code is nice & clean. However, when I click the buttons the Javascript validation doesn't seem to work.

    My suggestion would be to locate the eventListener for the form submit & add the preventDefault( ) method to stop the page refreshing.

    I could be wrong (I'm still a newbie) but worth having a look.

    1
    Etienne L.•95
    @etnlmyPosted over 3 years ago

    hey @philjacks , thanks a lot for your feedback :)

    You're right, I currently only validate fields when they loose the focus (on blur event). I guess I should also trigger the validation when I submit the form.

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord