Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#2098 closed enhancement (fixed)

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.

Change History (5)

comment:1 Changed 12 years ago by paul

Owner: changed from paul to iMarc

comment:2 Changed 12 years ago by kbwood

Owner: changed from iMarc to kbwood
Status: newassigned

comment:3 Changed 12 years ago by kbwood

Resolution: fixed
Status: assignedclosed

Implemented for all callbacks.

comment:4 Changed 12 years ago by (none)

Milestone: 1.2.2

Milestone 1.2.2 deleted

comment:5 Changed 11 years ago by paul

Milestone: 1.5
Note: See TracTickets for help on using tickets.