Opened 8 years ago

Closed 8 years ago

Last modified 8 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 8 years ago.
Datepicker HTML Test Case
cal.gif​ (127 bytes) - added by ihumanable 8 years ago.
Sample image used by test case

Download all attachments as: .zip

Change History (12)

Changed 8 years ago by ihumanable

Attachment: datepicker.html​ added

Datepicker HTML Test Case

Changed 8 years ago by ihumanable

Attachment: cal.gif​ added

Sample image used by test case

comment:1 Changed 8 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 8 years ago by rdworth

Milestone: 1.7 → 1.8

comment:3 Changed 8 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 8 years ago by rdworth

Resolution: → fixed
Status: assigned → closed

Fixed in r2303

comment:5 Changed 8 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 8 years ago by rdworth

Priority: major → critical

comment:7 Changed 8 years ago by rdworth

Milestone: 1.8 → 1.7.1

merged to 1.7.1 in r2327

comment:8 Changed 8 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 8 years ago by rdworth

Resolution: → fixed
Status: reopened → closed

fixed in r2350

comment:10 Changed 8 years ago by rdworth

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