Bug Tracker

Opened 5 years ago

Closed 5 years ago

Last modified 5 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 5 years ago by Rick Waldron

Owner: set to Rick Waldron
Status: newassigned

Thanks!

comment:2 Changed 5 years ago by Rick Waldron

Component: unfiledmisc
Milestone: None1.next
Priority: undecidedlow

comment:4 Changed 5 years ago by Rick Waldron

Resolution: fixed
Status: assignedclosed

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

More Details:

comment:5 Changed 5 years ago by dmethvin

Milestone: 1.next1.7.1
Note: See TracTickets for help on using tickets.