Bug Tracker

Ticket #10817 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

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

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

comment:1 Changed 3 years ago by rwaldron

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

Thanks!

comment:2 Changed 3 years ago by rwaldron

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

comment:4 Changed 3 years ago by Rick Waldron

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

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

More Details:

comment:5 Changed 3 years ago by dmethvin

  • Milestone changed from 1.next to 1.7.1
Note: See TracTickets for help on using tickets.