If you don't specify a #maxlength on a #type => 'textfield', it defaults to 128. For Notifications Team, this means that when you're using the autocomplete textfield to specific a comma-seperated list of new users to notify, it will only let you list up to 128 characters. Since you may have a site with very long user name (like full names), this is very limiting. On our site, it only allows you to notify 5-6 people!

Personally, I think this field shouldn't have a limit!

I'll submit a patch in a moment which makes this change.

Comments

dsnopek’s picture

Status: Active » Needs review
StatusFileSize
new554 bytes

Patch is attached!

dsnopek’s picture

StatusFileSize
new557 bytes

Attached is a patch that actually works! My previous patch made the HTML correct, but the Drupal validation would complain when the field contained more than 0 characters. :-) This one fixes the HTML and will validate.