Bug Tracker

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#10817 closed enhancement (fixed)

Change .bind() in ajax.js to .on()

Reported by: chrisr Owned by: Rick Waldron
Priority: low Milestone: 1.7.1
Component: misc Version: git
Keywords: Cc:
Blocked by: Blocking:

Description

In line 260 of ajax.js there is: 

return this.bind( o, f );

I believe, it should be .on() since .bind() is only wrapper for backward compatibility.

Also, there is .bind() in comment in line 70 in event.js ;)

Change History (5)

comment:1 Changed 10 years ago by Rick Waldron

Owner: set to Rick Waldron
Status: new → assigned

Thanks!

comment:2 Changed 10 years ago by Rick Waldron

Component: unfiled → misc
Milestone: None → 1.next
Priority: undecided → low

comment:4 Changed 10 years ago by Rick Waldron

Resolution: → fixed
Status: assigned → closed

Landing pull request 604. Updates bind to on in src/ajax.js. Fixes #10817.

More Details:

comment:5 Changed 10 years ago by dmethvin

Milestone: 1.next → 1.7.1
Note: See TracTickets for help on using tickets.