Bug Tracker

Modify

Ticket #10817 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 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 2 years ago by rwaldron

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

Thanks!

comment:2 Changed 2 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 2 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 2 years ago by dmethvin

  • Milestone changed from 1.next to 1.7.1

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.