#4240 closed bug (fixed)

Opened February 26, 2009 05:12PM UTC

Closed March 22, 2009 12:14PM UTC

Last modified June 12, 2009 12:38PM UTC

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)
  • cal.gif (0.1 KB) - added by ihumanable February 26, 2009 05:13PM UTC.

    Sample image used by test case

  • datepicker.html (0.9 KB) - added by ihumanable February 26, 2009 05:13PM UTC.

    Datepicker HTML Test Case

Change History (10)

Changed February 27, 2009 03:20PM UTC by jzaefferer comment:1

milestone: TBD → 1.7
priority: minor → major
summary: Datepicker Destroy Bug → Datepicker destroy affects other datepickers

Changed March 08, 2009 02:40PM UTC by rdworth comment:2

milestone: 1.7 → 1.8

Changed March 18, 2009 01:53AM UTC by rdworth comment:3

owner: → rdworth
status: new → assigned

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

Changed March 18, 2009 01:56AM UTC by rdworth comment:4

resolution: → fixed
status: assigned → closed

Fixed in r2303

Changed March 18, 2009 02:00AM UTC by rdworth comment:5

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.

Changed March 18, 2009 09:07AM UTC by rdworth comment:6

priority: major → critical

Changed March 19, 2009 03:30AM UTC by rdworth comment:7

milestone: 1.8 → 1.7.1

merged to 1.7.1 in r2327

Changed March 22, 2009 12:14PM UTC by rdworth comment:8

milestone: 1.7.1 → 1.8
resolution: fixed
status: closed → reopened
version: 1.6rc6 → 1.7.1

appendText is also affected

Changed March 22, 2009 12:14PM UTC by rdworth comment:9

resolution: → fixed
status: reopened → closed

fixed in r2350

Changed June 12, 2009 12:38PM UTC by rdworth comment:10

milestone: 1.8 → 1.7.2