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 comment:1
owner: | paul → iMarc |
---|
Changed January 09, 2008 03:18AM UTC by comment:2
owner: | iMarc → kbwood |
---|---|
status: | new → assigned |
Changed January 10, 2008 04:51AM UTC by comment:3
resolution: | → fixed |
---|---|
status: | assigned → closed |
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 comment:5
milestone: | → 1.5 |
---|