To maintain consistency with core webform components, the div surrounding the value in theme_webform_display_phonefield() should be removed. I will follow up with a patch to remove the prefix and suffix from the render array.

This results in cleaner output in webform email templates.

CommentFileSizeAuthor
#1 webform_phone-remove_prefix_and_suffix_from_field_display-1719660-1.patch427 bytesabarket

Comments

abarket’s picture

abarket commented
StatusFileSize
new webform_phone-remove_prefix_and_suffix_from_field_display-1719660-1.patch427 bytes

Here is the patch.

RadioActiv’s picture

RadioActiv commented
Assigned: Unassigned » RadioActiv
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, webform_phone-remove_prefix_and_suffix_from_field_display-1719660-1.patch, failed testing.

RadioActiv’s picture

RadioActiv commented
Status: Needs work » Closed (fixed)

Fix committed to trunk, tagged for 7.x-1.5.

arosboro’s picture

arosboro commented

Thanks RadioActiv, I helped Austin write this patch and I'm sure he'll be glad it was accepted