#8828 closed bug (fixed)
Button: Refresh does not re-enable disabled button.
| Reported by: | ezyang | Owned by: | tj.vantoll |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.9.2 |
| Component: | ui.button | Version: | 1.9.1 |
| Keywords: | regression haspatch | Cc: | |
| Blocked by: | Blocking: |
Description
See example: http://jsfiddle.net/U2Gmm/2/
It does, however, manage to capture enabled to disabled transition.
Change History (13)
comment:1 Changed 8 years ago by
| Component: | ui.core → ui.button |
|---|---|
| Status: | new → open |
| Summary: | Refresh does not re-enabled disabled button. → Button: Refresh does not re-enabled disabled button. |
comment:2 Changed 8 years ago by
| Summary: | Button: Refresh does not re-enabled disabled button. → Button: Refresh does not re-enable disabled button. |
|---|
comment:3 Changed 8 years ago by
| Resolution: | → notabug |
|---|---|
| Status: | open → closed |
comment:4 Changed 8 years ago by
Oops, sorry about the typo! It seems that the older version of jQuery UI handles it properly, but not the latest: http://jsfiddle.net/U2Gmm/3/
comment:5 Changed 8 years ago by
| Keywords: | regression added |
|---|---|
| Resolution: | notabug |
| Status: | closed → reopened |
Interestingly, this is a regression in 1.9.1
comment:6 Changed 8 years ago by
| Status: | reopened → open |
|---|
comment:7 Changed 8 years ago by
| Owner: | set to tj.vantoll |
|---|---|
| Status: | open → assigned |
Since this is a 1.9.1 regression this was likely caused by my change here - https://github.com/jquery/jquery-ui/commit/5e24a1ce4b337830b37511305a6ddefe797fd40c. I'll look into this.
comment:8 Changed 8 years ago by
So, just as a point of strategy, I'd like to see this NOT rolled back, but adding even more tests to keep both fixes, if possible.
comment:10 Changed 8 years ago by
| Keywords: | haspatch added |
|---|
Pull request - https://github.com/jquery/jquery-ui/pull/832.
comment:11 Changed 8 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Button: Fixing handling of the disabled options on refresh method calls. Fixed #8828 - Button: Refresh does not re-enable disabled button.
Changeset: 93abe02b6052143fac30393291da3fc254bde996
comment:12 Changed 8 years ago by
Button: Fixing handling of the disabled options on refresh method calls. Fixed #8828 - Button: Refresh does not re-enable disabled button. (cherry picked from commit 93abe02b6052143fac30393291da3fc254bde996)
Changeset: 8def77cb5cfefb733874d15a2294f02fc5ae78ae
@ezyang The example you provided has a JS error because there is no
refreshon$.fn. In order to callbutton'srefreshmethod you must use$('button').button('refresh'), which works fine - http://jsfiddle.net/tj_vantoll/hWkvs/.If you have an example that shows this issue without a JS error please comment and we can reinvestigate.