Create a form element that can be used in other (custom) forms too [#1931962]

Right now, you're only creating a Field. You could create a form API element (with hook_element_info()) that can be used by other forms, not just in Fields. The Field could then use that form API element and everybody would be so happy.

Comment	File	Size	Author
#3	timefield-form_element-1931962-3.patch	12.21 KB	tien.xuan.vo

Comments

Comment #1

wesnick commented 24 March 2013 at 20:46

I agree this would be nice to have. I think the element would be unwieldy, however, since it would require a huge number of properties, for example:

        '#separator' => ':',
        '#showLeadingZero' => FALSE,
        '#showMinutesLeadingZero' => TRUE,
        '#showPeriod' => TRUE,
        '#periodSeparator' => '',
        '#showHours' => TRUE,
        '#showMinutes' => TRUE,
        '#am_text' => 'AM',
        '#pm_text' => 'PM',
        '#minute_interval' => 5,
        '#showCloseButton' => FALSE,
        '#closeButtonText' => 'Done',
        '#showNowButton' => FALSE,
        '#nowButtonText' => 'Now',
        '#showDeselectButton' => FALSE,
        '#deselectButtonText' => 'Deselect',
        '#myPosition' => 'left top',
        '#atPosition' => 'left bottom',

Comment #2

rudiedirkx commented 24 March 2013 at 21:23

That's fine, isn't it? Use good, sensible defaults. Most of those options never have to change. Whether to display a 24 or 12 hour clock would be the most used probably.

Comment #3

tien.xuan.vo commented 27 September 2013 at 04:25

Status	File	Size
new	timefield-form_element-1931962-3.patch	12.21 KB

Here is the patch. I'm not good in form API and field API, so it just work.

This patch is for 1.0-alpha2.

Here is how to use it:

$form['my_timefield'] = array(
'#title' => t('Timefield'),
'#type' => 'timefield',
'#weight' => '0',
);

Comment #4

nicxvan commented 18 March 2016 at 17:02

This patchworks!

Comment #5

nicxvan commented 18 March 2016 at 17:03

Issue summary:	View changes
Status:	Active	» Reviewed & tested by the community

Comment #6

rv0 commented 24 May 2016 at 14:02

+1 for this functionality.

The project I need this sadly can't rely on a patched 3rd party module to work, so I'll roll my own time field for now.
But I'll considering switching if this code gets committed.