Skip to main content

Bug Tracker

Side navigation

#2098 closed enhancement (fixed)

Opened December 27, 2007 12:58PM UTC

Closed January 10, 2008 04:51AM UTC

Datepicker - onSelect should use apply() with input reference

Reported by: Stormtrooper Owned by: kbwood
Priority: minor Milestone: 1.2.2
Component: ui Version: 1.2.1
Keywords: datepicker onselect Cc:
Blocked by: Blocking:
Description

The onSelect callback of the Datepicker UI control should be bound to the input that triggered it and also pass the date variable to it as it does already.

for example (pseudo code):

// somewhere within the datepicker code ...

if (options.onSelect) {

options.onSelect.apply(the_input, [ date_var ]);

}

This way, the callback can reference 'this' as the input and can do more things depending on the input that triggered the calendar popup (useful when several date inputs are on a page). Adding this feature would not break previous versions since the date variable is still passed as the first argument to the callback.

Thanks.

Attachments (0)
Change History (3)

Changed December 27, 2007 03:16PM UTC by paul comment:1

owner: pauliMarc

Changed January 09, 2008 03:18AM UTC by kbwood comment:2

owner: iMarckbwood
status: newassigned

Changed January 10, 2008 04:51AM UTC by kbwood comment:3

resolution: → fixed
status: assignedclosed

Implemented for all callbacks.