Opened 14 years ago
Closed 14 years ago
#2415 closed bug (fixed)
draggable destroy() error
Reported by: | lideln | Owned by: | paul |
---|---|---|---|
Priority: | major | Milestone: | 1.2.4 |
Component: | ui | Version: | 1.2.3 |
Keywords: | draggable | Cc: | |
Blocked by: | Blocking: |
Description
When calling $('.myClass').draggable('destroy'); it crashes javascript on two cases :
- draggable() has not been called before to add drag capabilities to this object (I think it should just return quietly, not throw a JS error)
- draggable() has been called without specifying "handle". Adding "if (this.handle)" in the destroy() function fixes the bug.
Note: See
TracTickets for help on using
tickets.
this.handle always is available if the draggable is initialized, even if you didn't pass the option - however, I added a check so it doesn't break on non-draggables. Thanks!