As far as I can tell, when the datapicker is used to change the value of a date field, it does not cause a 'change' event in that field.
Normally this isn't an issue, but if people want to associate an action with the change, then it doesn't seem to work :-(

Maybe I've missed something?

(I'm trying to get dates working with editablefields, which uses the change event).

Cheers

Mark.

CommentFileSizeAuthor
#3 348166-3.patch1.58 KBjelle_s

Comments

markfoodyburton’s picture

markfoodyburton commented
Status: Active » Closed (fixed)

rubbish
my fault
sorry

theshanergy’s picture

theshanergy commented

Did you ever get this working? I'm having a related problem with editable fields - see my issue here http://drupal.org/node/1079446

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
Version: 6.x-2.x-dev » 7.x-2.x-dev
Status: Closed (fixed) » Active
StatusFileSize
new 348166-3.patch1.58 KB

We need this feature for Clientside Validation. Validation for fields fire on the change event, the datepicker changes the value of the field, but doesn't set the focus back on the field. Change events get triggered when a user clicks out of the input element and the value is changed, but since there is no focus on the element, no change event is triggered when the user clicks somewhere else, and thus no validation takes place. Attached is a patch to fix this.

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Active » Needs review
xatoo’s picture

xatoo commented

#3: 348166-3.patch queued for re-testing.

xatoo’s picture

xatoo commented
Status: Needs review » Needs work

After applying the patch I no longer get the timepicker popup. This message appears in the JS console instead.

Uncaught TypeError: Object 1360931430000 has no method 'getHours' jquery.timepicker.js:30
normalize jquery.timepicker.js:30
$.TimePicker._items jquery.timepicker.js:110
$.TimePicker.open jquery.timepicker.js:350
$.extend.open jquery.timepicker.js:211
(anonymous function) jquery.timepicker.js:247
p.event.dispatch jquery.min.js:2
g.handle.h jquery.min.js:2
p.event.trigger jquery.min.js:2
(anonymous function) jquery.min.js:2
p.extend.each jquery.min.js:2
p.fn.p.each jquery.min.js:2
p.fn.extend.trigger jquery.min.js:2
p.fn.(anonymous function) jquery.min.js:2
b.fn.extend.focus jquery-ui.min.js:11
(anonymous function) date_popup.js:60
p.event.dispatch jquery.min.js:2
g.handle.h

afmdsouza’s picture

afmdsouza commented
Version: 7.x-2.x-dev » 7.x-2.7
Issue summary: View changes

Thanks for patch, works fine with 7.x-2.7.