Hide the javascript innerHTML after error is corrected

Question

I have used the following javascript code to add inline error in my form. I want to error to be removed after the error is corrected after each field is corrected. I have created a single function for all the validations, so not able to use else and remove the error manually.

var emailpattern = /^[a-zA-Z][a-zA-Z0-9_]*(\.[a-zA-Z0-9_]+)*@[a-zA-Z0-9-]+(\.[a-zA-Z0-9-]+)*\.([a-zA-Z]{2,4})$/

var passwordpattern = /^(?=.*[a-z])(?=.*[A-Z])(?=.*[$@$!%*?&])[A-Za-z\d$@$!%*?&]{8,}$/;

function validateForm() {
  var x = document.forms["LoginForm"]["email"];
  if (x.value == "") {
    x.value = "";
    document.getElementById('pointemail').innerHTML = "Please enter the email id.";
    x.focus();
    return false;
  } else if (!emailpattern.test(x.value)) {
    x.value = "";
    document.getElementById('pointemail').innerHTML = "Please enter a valid email address.";
    x.focus();
    return false;
  }
  x = document.forms["LoginForm"]["password"];
  if (x.value == "") {
    x.value = "";
    document.getElementById('pointpassword').innerHTML = "Please enter the password.";
    x.focus();
    return false;
  } else if (!passwordpattern.test(x.value)) {
    x.value = "";
    document.getElementById('pointpassword').innerHTML = "Password should have minimum 8 characters, one upper case, one lower case and one special character";
    x.focus();
    return false;
  }
}

Just add an else block at the end for valid state

Rajesh Dixit
– Rajesh Dixit

2016-10-17 06:48:33 +00:00
Commented Oct 17, 2016 at 6:48 — Rajesh Dixit
– Rajesh Dixit, Commented Oct 17, 2016 at 6:48

priya_singh · Accepted Answer · 2016-10-17 06:48:58Z

1

Try This

var emailpattern = /^[a-zA-Z][a-zA-Z0-9_]*(\.[a-zA-Z0-9_]+)*@[a-zA-Z0-9-]+(\.[a-zA-Z0-9-]+)*\.([a-zA-Z]{2,4})$/

var passwordpattern = /^(?=.*[a-z])(?=.*[A-Z])(?=.*[$@$!%*?&])[A-Za-z\d$@$!%*?&]{8,}$/;

function validateForm() {
  var x = document.forms["LoginForm"]["email"];
  document.getElementById('pointemail').innerHTML = "";
  document.getElementById('pointpassword').innerHTML = "";

 //Add your if else here.
}

Hope it will be useful. :)

answered Oct 17, 2016 at 6:48

priya_singh

2,4881 gold badge19 silver badges33 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

priya_singh Over a year ago

It will be useful if you dont want to use else. It will first blank and after that check.

Álvaro Touzón Over a year ago

you are running it by the change event?

Shubham Sinha · Accepted Answer · 2016-10-17 06:51:09Z

0

Provide an extra else statement after elseif

else {
  document.getElementById('pointemail').innerHTML = '';
}

Similarly for pointPassword

answered Oct 17, 2016 at 6:51

Shubham Sinha

792 silver badges12 bronze badges

Collectives™ on Stack Overflow

Hide the javascript innerHTML after error is corrected

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related