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: rwaldron
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 rwaldron

  • Owner set to rwaldron
  • Status changed from new to assigned

Thanks!

comment:2 Changed 5 years ago by rwaldron

  • Component changed from unfiled to misc
  • Milestone changed from None to 1.next
  • Priority changed from undecided to low

comment:4 Changed 5 years ago by Rick Waldron

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

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 changed from 1.next to 1.7.1
Note: See TracTickets for help on using tickets.