Here is my code

document.onkeydown = function (a) {
    if (a.which == 13) {
        alert("Not Anymore");

    }
}

document.onkeydown = function (b) {
    if (b.which == 65) {
        auto();
    }
}

document.onkeydown = function (c) {
    if (c.which == 83) {
        auto2();

    }
}

Only the last snippet works can someone explain why this is happening check my website and you can see it isnt working when you press a but when you press b it is

Thanks, I appreciate the help and feedback

Here is my code

document.onkeydown = function (a) {
    if (a.which == 13) {
        alert("Not Anymore");

    }
}

document.onkeydown = function (b) {
    if (b.which == 65) {
        auto();
    }
}

document.onkeydown = function (c) {
    if (c.which == 83) {
        auto2();

    }
}

Only the last snippet works can someone explain why this is happening check my website and you can see it isnt working when you press a but when you press b it is

Thanks, I appreciate the help and feedback

• javascript
• onkeydown

Share Improve this question Follow edited Sep 14, 2015 at 3:36 Tushar 87.2k2121 gold badges163163 silver badges181181 bronze badges asked Sep 14, 2015 at 3:31 Duncan MilradDuncan Milrad 4111 gold badge11 silver badge44 bronze badges 2

• 2 The event handlers are overriden by the last, so only the last works – Tushar Commented Sep 14, 2015 at 3:33
• document.addEventListener(…); would be another option. – Sebastian Simon Commented Sep 14, 2015 at 3:39

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 5

You're binding the same event on the document multiple times. So, the later event handlers override the previous event handlers just like the functions with same name does. You need to bind only one event handler and use if... else in it.

You can use this

document.onkeydown = function (e) {
    if (e.which == 13) {
        alert("Not Anymore");
    } else if (e.which == 65) {
        auto();
    } else if (e.which == 83) {
        auto2();
    }
};

Also, use addEventListener instead of onkeydown.

document.addEventListener('keydown', function (a) {
    if (a.which == 13) {}
    ...
}, false);