Opened 10 years ago
Closed 9 years ago
#8975 closed bug (fixed)
Buttonset: refresh method causes JS error after adding new radio buttons
| Reported by: | gvas | Owned by: | tj.vantoll |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.11.0 |
| Component: | ui.button | Version: | 1.9.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
Adding a new radio button to a buttonset and invoking the buttonset's refresh method causes a JS exception. In contrast, adding new checkboxes works fine.
Tested in: Chrome 24, IE 7, IE 8
Operating system: Windows 7 Ultimate x64
Steps to reproduce:
- Initialize a buttonset
- Insert a new radio button below the buttonset
- Invoke the buttonset's refresh method
Expected:
The new radio button is added to the buttonset
Actual:
JS exception: "cannot call methods on button prior to initialization; attempted to call method 'widget'"
Change History (4)
comment:1 Changed 10 years ago by
| Milestone: | none → 1.11.0 |
|---|---|
| Owner: | set to tj.vantoll |
| Status: | new → assigned |
| Summary: | buttonset('refresh') causes exception after adding new radio buttons → Buttonset: refresh method causes JS error after adding new radio buttons |
comment:3 Changed 10 years ago by
Closed the pull request because this will have to be handled another way. The pull does have a failing test for this problem: https://github.com/jquery/jquery-ui/pull/888/files.
comment:4 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Button: Properly refresh button sets with new radio buttons
Fixes #8975 Ref gh-888
Changeset: 0059722b6b43c4985dbbd5f1494524442c12ddb0
Verified on master - http://jsfiddle.net/tj_vantoll/zQKV2/1/. I'll take a look at this one.