127
// this e works
document.getElementById("p").oncontextmenu = function(e) {
    e = e || window.event;
    var target = e.target || e.srcElement;
    console.log(target);
};

// this e is undefined
function doSomething(e) {
    e = e || window.event;
    var target = e.target || e.srcElement;
    console.log(target);
}
<p id="p" onclick="doSomething(e)">
    <a href="#">foo</a>
    <span>bar</span>
</p>

There are some similar questions have been asked.

But in my code, I'm trying to get child elements who's been clicked, like a or span.

So what is the correct way to pass event as an argument to event handler, or how to get event inside handler without passing an argument?

edit

I'm aware of addEventListener and jQuery, please provide a solution for passing event to inline event hander.

JohnK
  • 5,481
  • 6
  • 42
  • 69
user1643156
  • 3,657
  • 10
  • 34
  • 58
  • https://developer.mozilla.org/en/docs/DOM/EventTarget.addEventListener – Paul S. May 06 '13 at 17:54
  • 5
    Is there a good reason for using inline event handlers rather than switching to addEventListener? http://en.wikipedia.org/wiki/Unobtrusive_JavaScript – Xotic750 May 06 '13 at 17:56
  • 2
    @Xotic750 For me yes, no need to care about re-binding them manually everytime when dynamically loading or reloading html through ajax for example. – Wadih M. Aug 31 '19 at 20:07

4 Answers4

208

to pass the event object:

<p id="p" onclick="doSomething(event)">

to get the clicked child element (should be used with event parameter:

function doSomething(e) {
    e = e || window.event;
    var target = e.target || e.srcElement;
    console.log(target);
}

to pass the element itself (DOMElement):

<p id="p" onclick="doThing(this)">

see live example on jsFiddle.

You can specify the name of the event as above, but alternatively your handler can access the event parameter as described here: "When the event handler is specified as an HTML attribute, the specified code is wrapped into a function with the following parameters". There's much more additional documentation at the link.

JohnK
  • 5,481
  • 6
  • 42
  • 69
Akram Berkawy
  • 3,912
  • 2
  • 17
  • 26
  • 11
    (And for anyone wondering: Yes, this does work on Chrome, Firefox, etc., even though some [Firefox, for instance] don't have a global `event` object. It's because the context in which the DOM0 handler is called has an `event` object, even if [on some browsers] it isn't a global.) – T.J. Crowder May 06 '13 at 18:04
  • `this` refers to `p`, but I'm trying to get `a` or `span` – user1643156 May 06 '13 at 18:45
  • 3
    Passing ´event´ is the only way that I know, provided it is supported. Parsing ´this´ is of no use in this situation – Xotic750 May 06 '13 at 19:01
  • @AkramBerkawy that is it. I thought I could pass `e` the same way as in `ele.onclick = function(e)`, but it's `event`. Thanks. – user1643156 May 06 '13 at 19:02
  • 6
    @user1643156 That's very important. the parameter must be named exactly as "event" – The-null-Pointer- Feb 15 '18 at 10:37
  • 1
    Wadih M.'s answer is way better than this. This one is misleading (like others similar to it), making people believe that "(event)" parameter should be put into the function call, when, actually, JavaScript has the "event" parameter already and readily available inside the called function, so there is no need to declare it (it took me days to realise that it made no difference changing the variable name, because it was not really being passed). Also, the explanation given there dismisses any doubts about why JS works like that (maybe it shouldn't, but that's not on me to judge...) – Cyberknight Jun 03 '20 at 00:45
  • 1
    The reason the 'event' object can be used as an argument to the in-HTML inline event-handler is explained in https://developer.mozilla.org/en-US/docs/Web/Guide/Events/Event_handlers . – Panu Logic Aug 04 '20 at 02:59
19

Since inline events are executed as functions you can simply use arguments.

<p id="p" onclick="doSomething.apply(this, arguments)">

and

function doSomething(e) {
  if (!e) e = window.event;
  // 'e' is the event.
  // 'this' is the P element
}

The 'event' that is mentioned in the accepted answer is actually the name of the argument passed to the function. It has nothing to do with the global event.

Semra
  • 2,360
  • 24
  • 24
  • Good tip. When people will start adopting web components `call()` and `apply()` will prove essential in emulating data binding capabilities available in mainstream js frameworks. One extra trick is to do something similar to `Object.assign(this.querySelector('my-btn'), this)` inside a web component and voila, data binding. You can access any method of the parent web component class from the inline events `onclick="toggleBtnActive.call(this, true)"`. – Adrian Moisa Dec 09 '17 at 18:08
14

You don't need to pass this, there already is the event object passed by default automatically, which contains event.target which has the object it's coming from. You can lighten your syntax:

This:

<p onclick="doSomething()">

Will work with this:

function doSomething(){
  console.log(event);
  console.log(event.target);
}

You don't need to instantiate the event object, it's already there. Try it out. And event.target will contain the entire object calling it, which you were referencing as "this" before.

Now if you dynamically trigger doSomething() from somewhere in your code, you will notice that event is undefined. This is because it wasn't triggered from an event of clicking. So if you still want to artificially trigger the event, simply use dispatchEvent:

document.getElementById('element').dispatchEvent(new CustomEvent("click", {'bubbles': true}));

Then doSomething() will see event and event.target as per usual!

No need to pass this everywhere, and you can keep your function signatures free from wiring information and simplify things.

Wadih M.
  • 10,816
  • 6
  • 38
  • 52
  • 1
    At least on IE11, this is not true, There are no default arguments. – cskwg Feb 21 '20 at 05:58
  • 1
    @cskwg It does work for IE11 as well. It works on all major browsers as this is necessary for backwards compatibility. As I mentioned in the answer, only if the javascript function is fired from an event, which is the only scenario where an event exists, there will be an object already called "event" that you can readily access inside your function without having had to pass extra wiring to your function prototype. I just did a test on my machine with IE11 for an onclick event and and the 'event' variable contained a "PointerEvent object". – Wadih M. Mar 09 '20 at 20:40
0

here is how I do it to prevent user from copy/pasting invalid characters into input text field:

function validatePaste(el, e) {
  var regex = /^[a-z .'-]+$/gi;
  var key = e.clipboardData.getData('text')
  if (!regex.test(key)) {
    e.preventDefault();
    return false;
  }
}

This function is located inside tags and it is called like:

<input type="text" onpaste="validatePaste(event)">     

                    
pixel
  • 6,765
  • 12
  • 60
  • 101