#5945 closed bug (fixed)
Button: Disabled buttons still allow clicks
| Reported by: | spudly | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.8.14 |
| Component: | ui.button | Version: | 1.8.4 |
| Keywords: | click event disabled | Cc: | |
| Blocked by: | Blocking: |
Description (last modified by )
When you disable a button widget, the click events still fire. I expected events to somehow be turned off somehow when the button is disabled, like native button elements.
This only occurs on elements that can't be disabled via the disabled attribute.
Change History (17)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
| Milestone: | TBD → 1.9 |
|---|
comment:3 Changed 13 years ago by
| Description: | modified (diff) |
|---|---|
| Status: | new → open |
| Summary: | Disabled buttons still fire click event → Button: Disabled buttons still allow clicks |
comment:6 follow-up: 11 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | open → closed |
Button: stop disabled button from firing click events. fixes #5945. Behavior was inconsisten between BUTTON and A elements.
Changeset: a7f1659cdc99222a733ed2edbd7d00dc57c30227
comment:7 Changed 12 years ago by
Merge pull request #207 from eikes/bug-5945
Button: stop disabled button from firing click events. fixes #5945.
Changeset: af5b5b19483a0a18abe1fbb43cb534a5485f0804
comment:8 Changed 12 years ago by
Button: stop disabled button from firing click events. fixes #5945. Behavior was inconsisten between BUTTON and A elements. (cherry picked from commit a7f1659cdc99222a733ed2edbd7d00dc57c30227)
Changeset: 4c218eeb0a4e6b06d055fcfe0b45ff034ecbfa43
comment:9 Changed 12 years ago by
| Milestone: | 1.9 → 1.8.13 |
|---|
comment:11 follow-up: 12 Changed 12 years ago by
Replying to Eike Send:
Button: stop disabled button from firing click events. fixes #5945. Behavior was inconsisten between BUTTON and A elements.
Changeset: a7f1659cdc99222a733ed2edbd7d00dc57c30227
The behavior of disabled anchor elements does not match previous jQuery UI versions. See examples.
- jQuery UI 1.8.2: http://jsfiddle.net/JCapriotti/Fw8NC/
- jQuery UI 1.8.13: http://jsfiddle.net/JCapriotti/zRNV8/
Also, disabling the hyperlink functionality of anchor elements requires the element to have a click handler that returns false. I believe it has been this way for some time. Is there any way to remove this requirement?
comment:12 follow-up: 13 Changed 12 years ago by
| Milestone: | 1.8.13 → 1.9 |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
Yeah, that should be preventing the default in addition to stopping propagation.
comment:13 follow-up: 14 Changed 12 years ago by
Replying to scott.gonzalez:
Yeah, that should be preventing the default in addition to stopping propagation.
If someone managed to fix it; could it be added to 1.8.14? If so, when would that fix need to happen by?
comment:14 Changed 12 years ago by
Replying to JasonC:
If someone managed to fix it; could it be added to 1.8.14? If so, when would that fix need to happen by?
Yes. We don't set dates, but we release about once per month.
comment:15 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Button: adding event.preventDefault. Fixes #5945 - Disabled buttons still allow clicks
Changeset: 597a313ab67bc10627d9c17c941776514ea5bd7c
comment:16 Changed 12 years ago by
Button: adding event.preventDefault. Fixes #5945 - Disabled buttons still allow clicks (cherry picked from commit 597a313ab67bc10627d9c17c941776514ea5bd7c)
Changeset: 3735bab147b663767d2268b90ac27234b1c19fbf
comment:17 Changed 12 years ago by
| Milestone: | 1.9 → 1.8.14 |
|---|
#6855 is a duplicate of this ticket.