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.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | autocomplete-maxlength-1994882-2.patch | 557 bytes | dsnopek |
| #1 | autocomplete-maxlength-1994882.patch | 554 bytes | dsnopek |
Comments
Comment #1
dsnopekPatch is attached!
Comment #2
dsnopekAttached 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.