Comments on: jQuery: passing elements to event handlers

By: Jquery for Beginners: Events Part 1 « Asim Sajjad

Jquery for Beginners: Events Part 1 « Asim Sajjad — Sun, 21 Dec 2008 13:13:07 +0000

[...] Passing Event Data: Now here is the code to bind the event with object using the bind event handler and also pass the data as well.The code below is taken from Infovore [...]

By: Tom

Tom — Thu, 07 Aug 2008 09:25:02 +0000

Sorry you feel that way, Sam. I felt that Vincent’s code stood well on its own: it’s expressive, it’s better code than mine, and it works. I probably should have said “yes, that works”, but I didn’t actually have time to test it. You know, due to being very busy, and my code also working (even if it’s not as expressive).

So, you know: Vincent’s code is good, it works, it’s better than mine; it feels like a nice footnote on the whole endeavour. Anyhow, now I’ve replied. Happy?

If, of course, you’re a spambot, I’m wasting my time writing back to you. But it’s worth a pop.

By: Sam

Sam — Wed, 06 Aug 2008 20:31:25 +0000

I’m getting tired of seeing blog posts where the author doesn’t reply to good comments and questions.

Great way to make sure whatever readers you do have, stop.

By: Vincent Robert

Vincent Robert — Fri, 10 Aug 2007 12:46:26 +0000

You actually wanted to toggle the class for every link except the first one (which you toggle twice here). Why not just implement directly what you meant ? Here is a simpler version :

$(document).ready(function() {
	$("div.container h3 a")	
		// Add the event handle "onclick"
		.click(function() {
			$(this).parent()
				.next()
					.toggle()
				.end()
				.toggleClass("open")
				;
		})
		// Trigger it manually for every link exept the first one
		.filter(":gt(0)")
			.click()
		.end()
		;
	};
});

I didn't tested but it should work. Hope this helps.

By: Linkdumpingâ€”Time Wasted on Binaries

Linkdumpingâ€”Time Wasted on Binaries — Mon, 06 Aug 2007 23:10:07 +0000

[...] Here are parts of todays finds, jotted down for later reference: GZIP compression, HTTP Caching, the Pareto principle (plus the wikipedia equivalent), passing elements to event handlers with jQuery, CSS Compress for WordPress, This day in type, Pingdom, We love TXP, Jon Hicks’ google reader theme, lovely inspiring typography works by Craig Ward… [...]

By: malsup

malsup — Mon, 06 Aug 2007 17:13:52 +0000

Tom, You make a great point about dealing with function arguments based on context. But it is possible to use a single fn for the toggling, although it's somewhat less intuitive:


$(document).ready(function() {
    var $links = $("div.container h3 a");
    $links.click(toggleArchiveLinks);
    toggleArchiveLinks($links[0]);
});

function toggleArchiveLinks(event) {
    var element = event.href ? event : this;
    var $p = $(element).parent();
    $p.toggleClass("open").next().toggle();
    return false;
}