Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
With latest webform an input field of type email will not be triggered because the javascript is short circuited. Here is a patch to fix this.
Comment | File | Size | Author |
---|---|---|---|
#9 | add-email-fields-revised-1717428-9.patch | 704 bytes | ryanblyth |
#8 | add-email-field-1717428-8.patch | 8.44 KB | ryanblyth |
allow_email_fields.patch | 546 bytes | jerodfritz |
Comments
Comment #1
bfodeke CreditAttribution: bfodeke commentedI was unable to get your patch to work, I had to do the following in order to get it to work.
** edit **
Sorry, my solution is for the D7 version of this module, I don't know if it would help on D6
Comment #2
ksavoie CreditAttribution: ksavoie commentedHad same issue, bfodeke's suggestion worked for me in D7.
Thanks!
Comment #3
iwuv CreditAttribution: iwuv commentedI'm having this issue on Drupal 6 with the 1.3 version. Email field is unaffected by the Javascript.
Comment #4
rtrubshaw CreditAttribution: rtrubshaw commentedThe maintainers of jQuery closed ticket #9132 with the status "wontfix" for various good reasons, so the patch as given (adding input:email pseudo-selector) won't work.
The suggestion by bfodeke is the one that the jQuery maintainer's endorse (AFAICT).
It may be worth adding input[type="other HTML5 input type"] for all the other HTML5 input types that render as textboxes
Comment #5
iwuv CreditAttribution: iwuv commentedI've confirmed that the code in post #1 by bfodeke fixes this issue in the 1.3 version of Compact Forms on Drupal 6.
Here's how to do it. Add "input[type=email]" (no quotes) to line 22 of compact_forms.js as shown in bfodeke's example. And you're done.
Comment #6
jorisx CreditAttribution: jorisx commentedCheck post form #1 /#5 works perfect, also for D7
Can that be checkin for next release ?
:-)
Comment #7
ddorian CreditAttribution: ddorian commentedPost in #1 did it for me in d7.
Comment #8
ryanblyth CreditAttribution: ryanblyth commentedIn D7 #1 works for me too, thanks bfodeke. Here is a D7 patch using the solution from #1.
Comment #9
ryanblyth CreditAttribution: ryanblyth commentedUse this patch instead of #8. Recreated patch to remove unnecessary style text.
Comment #10
joachim CreditAttribution: joachim commentedThis is a duplicate of #1459346: Email labels in Webform 3.16+ not processed by compact form, where there is also a patch.