Bug Tracker

Modify ↓

Ticket #10817 (closed enhancement: fixed)

Opened 19 months ago

Last modified 16 months 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 19 months ago by rwaldron

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

Thanks!

comment:2 Changed 19 months 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 19 months 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 16 months 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.

Attachments ↑
 
Note: See TracTickets for help on using tickets.