Side navigation
#10817 closed enhancement (fixed)
Opened November 17, 2011 05:03PM UTC
Closed November 17, 2011 08:02PM UTC
Last modified March 02, 2012 04:29PM UTC
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 ;)
Attachments (0)
Change History (5)
Changed November 17, 2011 05:05PM UTC by comment:1
| owner: | → rwaldron |
|---|---|
| status: | new → assigned |
Changed November 17, 2011 05:06PM UTC by comment:2
| component: | unfiled → misc |
|---|---|
| milestone: | None → 1.next |
| priority: | undecided → low |
Changed November 17, 2011 05:21PM UTC by comment:3
Changed November 17, 2011 08:02PM UTC by comment:4
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
Landing pull request 604. Updates bind to on in src/ajax.js. Fixes #10817.
More Details:
Changeset: 327bc07c54526979527f9622c99efdcc8c715f7a
Changed March 02, 2012 04:29PM UTC by comment:5
| milestone: | 1.next → 1.7.1 |
|---|
Thanks!