Once #2895166: Add elements to show datetime and date using jQuery calendars popup is complete, we need widgets to actually show the popup element.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 2895167-2.patch | 8.09 KB | hussainweb |
Once #2895166: Add elements to show datetime and date using jQuery calendars popup is complete, we need widgets to actually show the popup element.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 2895167-2.patch | 8.09 KB | hussainweb |
Comments
Comment #2
hussainwebInitial patch with the popup widget. The popup calendar works but the date is not stored correctly.
Comment #3
anas_maw commentedShould we create 2 more widget for taarikh_date element?
Comment #4
hussainweb@Anas_maw, I have one more widget planned (already in the patch) for a simple select list. Which other widget would you need?
Comment #5
anas_maw commented@hussainweb in src/element there is to type of form element taarikh_date and taarikh_datetime
That what i mean.
Comment #6
hussainwebAh, so if you edit the field settings of a regular datetime field and set it to store Date only, only Date will be shown. Technically, taarikh_datetime is still used but it will just contain a taarikh_date directly, and time won't be shown. This is already working.
Comment #7
anas_maw commentedThen we should remove taarikh_date form element, no need for it.
Comment #8
hussainwebIt actually mirrors the current datetime and date form elements in the core. I think it is helpful to keep it to keep the API parity between core and this module. This way, we will also be able to reuse from core as much as possible.
Comment #10
hussainwebPushed changes for the default widget