jQuery timer not working on input

Question

I am trying to fire an event 3 seconds after the user has stopped typing to check the input to see if it meets the minimum length requirement. If it does, then show as valid and if less than 14 characters show as invalid/error. I want onkeydown to clear timer, check for positive validation, and if it fails to start another timer.

var typingTimer;                //timer identifier
var doneTypingInterval = 3000;  //time in ms, 3 second for example

// on keydown, start the countdown
$('#contact_phone').keydown(function(){
clearTimeout(typingTimer);
if ($('#contact_phone').val().length == 14) {
    typingTimer = setTimeout(doneTyping, doneTypingInterval);
}
});

//user is "finished typing," do something
function doneTyping () {
$('#contact_phone').keyup(function() {
    if ($(this).val().length == 14) {
        $(this).removeClass('whiteBorder').removeClass('error').addClass('valid');
    } else {
        $(this).removeClass('whiteBorder').removeClass('valid').addClass('error');
    }
});
}

Is it working for you? what problems are you having (if any?) — user400654
– user400654, Commented Dec 2, 2013 at 21:53

Sushanth -- · Accepted Answer · 2013-12-02 22:02:04Z

1

You are binding the keyup event every single time the timer expires.

You need not bind both the events to the input. ' Remove the keydown and replace it with keyup event.

var typingTimer; //timer identifier
var doneTypingInterval = 3000; //time in ms, 5 second for example

// on keydown, start the countdown
$('#contact_phone').keyup(function () {
    clearTimeout(typingTimer);
    typingTimer = setTimeout(doneTyping, doneTypingInterval);
});

//user is "finished typing," do something
function doneTyping() {
    var $this = $('#contact_phone');

    if ($this.val().length == 14) {
        $this.removeClass('whiteBorder').removeClass('error').addClass('valid');
    } else {
        $this.removeClass('whiteBorder').removeClass('valid').addClass('error');
    }
}

Check Fiddle

edited Dec 2, 2013 at 22:02

answered Dec 2, 2013 at 21:54

Sushanth --

55.8k9 gold badges70 silver badges109 bronze badges

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

2 Comments

Sushanth -- Over a year ago

@user2736472.. Removed the 2nd event that is not required. Also changed the event name from onkeyup to keyup

Sushanth -- Over a year ago

Glad to have helped :)

charlietfl · Accepted Answer · 2013-12-02 21:57:40Z

0

Problem is you are waiting until after user stops typing to bind the keyup event. Since user has stopped typing...this event will never fire.

Try:

// on keydown, start the countdown
$('#contact_phone').keydown(function () {
    var $input= $(this);
    clearTimeout(typingTimer);
    if ($('#contact_phone').val().length == 14) {
        typingTimer = setTimeout(function () {
            doneTyping($input);
        }, doneTypingInterval);
    }
});

//user is "finished typing," do something
function doneTyping($input) {

        if ($input.val().length == 14) {
            $input.removeClass('whiteBorder').removeClass('error').addClass('valid');
        } else {
           $input.removeClass('whiteBorder').removeClass('valid').addClass('error');
        }

}

answered Dec 2, 2013 at 21:57

charlietfl

172k14 gold badges125 silver badges154 bronze badges

1 Comment

charlietfl Over a year ago

create a demo in jsfiddle.net

Collectives™ on Stack Overflow

jQuery timer not working on input

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related