Bug Tracker

Modify

Ticket #1253 (closed enhancement: fixed)

Opened 7 years ago

Last modified 6 years ago

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:
Blocking: Blocked by:

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

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

Change History

Changed 7 years ago by brandon

Patch

comment:2 Changed 7 years ago by brandon

  • need changed from Review to Test Case

Changed 7 years ago by john

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

Changed 7 years ago by john

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

comment:3 Changed 7 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 7 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: ↓ 6 Changed 6 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 6 years ago by brandon

Replying to brandon:

That link should be:  event_enhancements branch.

comment:7 Changed 6 years ago by cloudream

fixs in r4162

comment:8 Changed 6 years ago by davidserduke

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

Fixed as cloudream said again. :)

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.