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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markfoodyburton’s picture

Status: Active » Closed (fixed)

rubbish
my fault
sorry

theshanergy’s picture

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

Version: 6.x-2.x-dev » 7.x-2.x-dev
Status: Closed (fixed) » Active
FileSize
1.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

Status: Active » Needs review
xatoo’s picture

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

xatoo’s picture

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

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

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