binding predefined functions to Jquery events

Question

I have a working script to apply transformations to SVG so that the image is draggable. The script works when I use the attributes of the html element.

<g id="all" onmousedown="startMove(evt)" onmousemove="moveIt(evt)" onmouseup="endMove(evt)">

However I would like to bind the events with Jquery something like below and wonder where I am going wrong with the code below

$('svg > g').mousedown(startMove(evt));
$('svg > g').mousemove(moveIt(evt));
$('svg > g').mouseup(endMove(evt));

The event expects a function but you're passing the return value of the function. Functions are objects too, you should pass a reference not execute it. So el.event(fn) not el.event(fn()) — elclanrs
– elclanrs, Commented Sep 4, 2013 at 8:22

Johan · Accepted Answer · 2013-09-04 08:20:23Z

1

Try this:

$('svg > g').mousedown(function(evt){
    //do stuff
});

or even

$('svg > g').mousedown(startMove);

function startMove(evt){
    //do stuff
}

startMove references the actual function, while startMove() uses it's return value.

answered Sep 4, 2013 at 8:20

Johan

35.4k62 gold badges191 silver badges309 bronze badges

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

2 Comments

Peter Saxton Over a year ago

I think the second one only works when you have no arguments to pass to the function

Arun P Johny Over a year ago

@PeterSaxton in this case it should work because it is expecting the event as the argument

Dale · Accepted Answer · 2013-09-04 08:20:02Z

0

I believe you have to do it like so:

$('svg > g').mousedown(function(evt){
    startMove(evt);
});

answered Sep 4, 2013 at 8:20

Dale

10.5k23 silver badges34 bronze badges

Collectives™ on Stack Overflow

binding predefined functions to Jquery events

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