Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.