Bug Tracker

Opened 9 years ago

Closed 8 years ago

#1253 closed enhancement (fixed)

Allow binding to multiple events simultaneously with .bind()

Reported by: daemach Owned by:
Priority: major Milestone: 1.2.2
Component: event Version: 1.1.2
Keywords: Cc:
Blocked by: Blocking:

Description

I was just reading in this cool book "Pro Javascript Techniques" about how it's a good idea to bind events to both keypress and mouse events for accessibility. On that note, I'd like to ask again if you can implement binding to multiple events with bind.

$('#myinput').bind("focus mouseover",function(){doit();})

Attachments (3)

event.js.diff (1.1 KB) - added by brandon 9 years ago.
Patch
event.js.diff2 (910 bytes) - added by john 9 years ago.
Here's my stab at it - reduce the number of calls to jQuery.event.
event.js2.diff (910 bytes) - added by john 9 years ago.
It didn't recognize it as a patch, trying again.

Download all attachments as: .zip

Change History (10)

Changed 9 years ago by brandon

Patch

comment:2 Changed 9 years ago by brandon

  • need changed from Review to Test Case

Changed 9 years ago by john

Here's my stab at it - reduce the number of calls to jQuery.event.

Changed 9 years ago by john

It didn't recognize it as a patch, trying again.

comment:3 Changed 9 years ago by john

  • Milestone changed from 1.1.3 to 1.2

Ok, we actually landed this similar functionality for classes in 1.1 (#172), so we should hold this off until 1.2. I've re-tagged the milestone to reflect that.

comment:4 Changed 9 years ago by brandon

This latest patch doesn't work for the one method.

I've created a plugin/extension until we can get this into the core. http://blog.brandonaaron.net/2007/06/05/bind-multiple-events-simultaneously-with-jquery/

comment:5 follow-up: Changed 8 years ago by brandon

  • Milestone changed from 1.2 to 1.2.2

I've started work on this in the branch. I've moved the logic directly into the add and remove event functions.

comment:6 in reply to: ↑ 5 Changed 8 years ago by brandon

Replying to brandon:

That link should be: event_enhancements branch.

comment:7 Changed 8 years ago by cloudream

fixs in r4162

comment:8 Changed 8 years ago by davidserduke

  • Resolution set to fixed
  • Status changed from new to closed

Fixed as cloudream said again. :)

Note: See TracTickets for help on using tickets.