@@ -15,13 +15,23 @@ (function($) { $.fn.labelize = function(hoverClass) { - var containers = $(this).filter(':has(input)'); + function labelClickEvent() { + // remove encompassing event (prevents jQuery recursion error in 1.3.x) + $(this).unbind('click', labelClickEvent); + + // call .click on owned input + $('input', this).click(); + + // re-apply the event after we're done + $(this).click(labelClickEvent); + } - // Apply cursor attribute to containers - $(containers).css('cursor', 'pointer'); + var containers = $(this).filter(':has(input)'); - // Apply click event to container - containers.click($.fn.labelize.labelClickEvent); + // Apply cursor attribute and onclick event to containers + $(containers) + .css('cursor', 'pointer') + .click(labelClickEvent); // Apply optional hoverClass if (hoverClass) { @@ -34,17 +44,13 @@ // input element; this makes sure click() events don't fire twice $('input', this) .mouseover(function() { - $(containers).unbind('click', $.fn.labelize.labelClickEvent); + $(containers).unbind('click', labelClickEvent); }) .mouseout(function() { - $(containers).click($.fn.labelize.labelClickEvent); + $(containers).click(labelClickEvent); }); return this; } - $.fn.labelize.labelClickEvent = function() { - $('input', this).click(); - } - })(jQuery); jquery.labelize.js 4410c9a69b008d3620bf6cad82855e3a501c90ae bentlegen ben@benlog.org http://github.com/bentlegen/labelize/commit/e973a34e8338b5f618c0e860b27ac6927047a73b e973a34e8338b5f618c0e860b27ac6927047a73b 2009-01-24T09:27:17-08:00 2009-01-24T09:26:09-08:00 Now works w/ 1.3.1 (cherry picked from commit da7170b8aaf6603535951ed43ee165dec307bd50) d92168b58ece4f0a4e966cfec5d914726cc09d35 bentlegen ben@benlog.org