Pretty straight forward issue, the Elements module adds the HTML 5 email field to Drupal, it would be nice to use this as the widget for the Email field if it is available.

Patch incoming.

CommentFileSizeAuthor
#4 interdiff.txt1.39 KBbgilhome
#4 email-support_elements-2500865-4.patch1.63 KBbgilhome
#3 email-support_elements-2500865-3.patch758 bytesbgilhome
#1 emailfield-2500865-1.patch1.44 KBDeciphered
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Deciphered’s picture

Deciphered CreditAttribution: Deciphered at Realityloop for The Wilderness Society commented
Status: Active » Needs review
FileSize
emailfield-2500865-1.patch1.44 KB
zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie at Ny Media AS commented
Status: Needs review » Needs work

I think there is no need to create new HTML5 widget here. I would opt for just changing field #type to 'email_emailfield' if Elements module is installed. It is like that for Telephone module.

But if you would prefer your approach more than the one I've suggested above, it could be easily done by new contrib micro-module (i.e. email_elements or elements_email). No need to wait for module maintainers to commit your patch.

bgilhome’s picture

bgilhome CreditAttribution: bgilhome commented
FileSize
email-support_elements-2500865-3.patch758 bytes

Here's a patch to check for 'elements' module and set form element type accordingly.

bgilhome’s picture

bgilhome CreditAttribution: bgilhome commented
FileSize
email-support_elements-2500865-4.patch1.63 KB
interdiff.txt1.39 KB

Actually we should also use only one element_validate callback if the elements module is installed - patch attached & interdiff to previous patch.

jackalope’s picture

jackalope CreditAttribution: jackalope commented

Thanks for the patch @bgilhome! I just encountered this need and your patch seems to be working well for adding HTML5 support via the Elements module.