Add support for textareas

Same issue
Subscribing

As far as I can tell from the code, textareas were never supported.

However, adding support for them might be doable. Patches are welcome.

Hi

The following fix works for me :
compact_forms.js (line 17)
change
if ($field.attr('type') != 'text' && $field.attr('type') != 'password') {
into
if ($field.attr('type') != 'text' && $field.attr('type') != 'textarea' && $field.attr('type') != 'password') {

Regards

Laurent

Sorry, I'd be really surprised when that works, because that's a bit nonsense.

$field.attr('type') != 'text' checks for the HTML attribute "type". Textareas do not have a "type" attribute. Instead, they are TEXTAREA elements and not INPUT elements.

Please test attached patch.

Hi Sun

Your patch is also working fine.
Mine is working because a textarea is considered as text type (see 'Attributes' section in http://www.w3.org/TR/2000/WD-DOM-Level-1-20000929/level-one-html.html#ID...).

Thanks

Laurent

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.