Ticket #2098 (closed enhancement: fixed)
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: | |
| Blocking: | Blocked by: |
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
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
