Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#4240 closed bug (fixed)

Datepicker: destroy, disable, and enable methods affect sibling datepickers

Reported by: ihumanable Owned by: rdworth
Priority: critical Milestone: 1.7.2
Component: ui.datepicker Version: 1.7.1
Keywords: Cc:
Blocked by: Blocking:

Description

When $(...).datepicker('destroy'); is issued all image buttons (buttons defined during initialization with a buttonImage and buttonImageOnly: true) for all datepickers are removed.

An example is attached, clicking the init startdate and init enddate will place two images on the screen in the correct position and wired properly. Clicking either of the remove buttons, removes both of the images instead of just the one associated with the datepicker being destroyed.

Attachments (2)

datepicker.html​ (954 bytes) - added by ihumanable 11 years ago.
Datepicker HTML Test Case
cal.gif​ (127 bytes) - added by ihumanable 11 years ago.
Sample image used by test case

Download all attachments as: .zip

Change History (12)

Changed 11 years ago by ihumanable

Attachment: datepicker.html​ added

Datepicker HTML Test Case

Changed 11 years ago by ihumanable

Attachment: cal.gif​ added

Sample image used by test case

comment:1 Changed 11 years ago by Jörn Zaefferer

Milestone: TBD → 1.7
Priority: minor → major
Summary: Datepicker Destroy Bug → Datepicker destroy affects other datepickers

comment:2 Changed 11 years ago by rdworth

Milestone: 1.7 → 1.8

comment:3 Changed 11 years ago by rdworth

Owner: set to rdworth
Status: new → assigned

Added visual test ​http://jquery-ui.googlecode.com/svn/trunk/tests/visual/datepicker/datepicker_ticket_4240.html in r2302

comment:4 Changed 11 years ago by rdworth

Resolution: → fixed
Status: assigned → closed

Fixed in r2303

comment:5 Changed 11 years ago by rdworth

Summary: Datepicker destroy affects other datepickers → Datepicker: destroy, disable, and enable methods affect sibling datepickers

This also affects 'enable' and 'disable' methods. For all three (destroy, enable, disable) it only affects sibling datepickers. So as a work-around, you can put each datepicker input or div in a wrapper element.

comment:6 Changed 11 years ago by rdworth

Priority: major → critical

comment:7 Changed 11 years ago by rdworth

Milestone: 1.8 → 1.7.1

merged to 1.7.1 in r2327

comment:8 Changed 11 years ago by rdworth

Milestone: 1.7.1 → 1.8
Resolution: fixed
Status: closed → reopened
Version: 1.6rc6 → 1.7.1

appendText is also affected

comment:9 Changed 11 years ago by rdworth

Resolution: → fixed
Status: reopened → closed

fixed in r2350

comment:10 Changed 11 years ago by rdworth

Milestone: 1.8 → 1.7.2
Note: See TracTickets for help on using tickets.