#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 6 years ago by
| Owner: | set to Rick Waldron |
|---|---|
| Status: | new → assigned |
comment:2 Changed 6 years ago by
| Component: | unfiled → misc |
|---|---|
| Milestone: | None → 1.next |
| Priority: | undecided → low |
comment:4 Changed 6 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Landing pull request 604. Updates bind to on in src/ajax.js. Fixes #10817.
More Details:
comment:5 Changed 5 years ago by
| Milestone: | 1.next → 1.7.1 |
|---|
Note: See
TracTickets for help on using
tickets.
Thanks!