Hi,

Do you think it can be possible to integrate the jQuery UI datepicker widget instead of the custom one ?

Thanks

CommentFileSizeAuthor
#6 makemeeting-datepicker-1356708-6.patch3.76 KBKi
#4 patch.diff3.03 KBSebCorbin
#2 patch.diff2.66 KBSebCorbin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SebCorbin’s picture

SebCorbin CreditAttribution: SebCorbin commented
Assigned: Unassigned » SebCorbin
Status: Active » Needs work

This is part of the 7.x-2.x rewrite

SebCorbin’s picture

SebCorbin CreditAttribution: SebCorbin commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Needs work » Needs review
FileSize
patch.diff2.66 KB

Status: Needs review » Needs work

The last submitted patch, patch.diff, failed testing.

SebCorbin’s picture

SebCorbin CreditAttribution: SebCorbin commented
Status: Needs work » Needs review
FileSize
patch.diff3.03 KB

Ignored datepicker in tests

The last submitted patch, patch.diff, failed testing.

Ki’s picture

Ki CreditAttribution: Ki commented
Issue summary: View changes
FileSize
makemeeting-datepicker-1356708-6.patch3.76 KB

I took SebCorbin's patch and added some revision to use datepicker.

- Set the string format to be yyyy-mm-dd, which I think can be understood by many without confusion.
- Removed timezone changing in _makemeeting_date_timestamp(), which caused to display incorrect date.
- Added sorting in _makemeeting_sort_choices() for string date.