#2959 closed bug (wontfix)
attr() broken in jQuery 1.2.6?
| Reported by: | phil | Owned by: | flesler |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.3 |
| Component: | core | Version: | 1.2.5 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
I'm having a problem using element.attr() ever since I upgraded to jquery 1.2.6. I know this issue doesn't exist in 1.2.4 (I've tested it).
Basically, calling element.attr(name, value) doesn't appear to do anything. I've attached a minimal HTML file that illustrates this issue. The page consists of a one-button form, and a separate button marked 'configure'. An alert will pop up if the behavior is incorrect.
Attachments (2)
Change History (5)
Changed 12 years ago by
| Attachment: | jquery-attr-bug.html added |
|---|
comment:1 Changed 12 years ago by
Changed 12 years ago by
| Attachment: | attr-form.diff added |
|---|
comment:2 Changed 12 years ago by
| need: | Review → Commit |
|---|---|
| Owner: | set to flesler |
| Priority: | major → minor |
| Status: | new → assigned |
The patch about should modify attr() so that it supports this situation. It won't be applied for now for 2 reasons:
1- The demo you showed doesn't work cross browser. As far as I tested, IE doesn't fire handlers set as strings with javascript. So there's no point in supporting that. 2- It needs a lot more of testing. Our test runner doesn't fail with this patch, but as attr() went to many changes lately, we'll give it special attention before modifying it again.
Thanks for reporting.
comment:3 Changed 12 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | assigned → closed |
This test case isn't good enough, but the issue is indeed valid. I'll close this ticket and move the patch to #3113 that provides a consistent problem.
Oh, I've only tested this with Firefox on Mac OS X 10.5.3, if it matters.