Skip to main content

Search and Top Navigation

#2098 closed enhancement (fixed)

Opened December 27, 2007 12:58PM UTC

Closed January 10, 2008 04:51AM UTC

Last modified February 26, 2009 11:22AM UTC

Datepicker - onSelect should use apply() with input reference

Reported by: Stormtrooper Owned by: kbwood
Priority: minor Milestone: 1.5
Component: ui.core 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 (5)

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.

Changed May 24, 2008 03:39AM UTC by comment:4

milestone: 1.2.2

Milestone 1.2.2 deleted

Changed February 26, 2009 11:22AM UTC by paul comment:5

milestone: → 1.5