If you have a large number of forms that you would like to restrict BeautyTips text popups to, you are limited by the number of form_id's that will fit in a text field with maxlength of 128. I think the solution here is to use a textarea instead of a textfield to hold the form_id's.

CommentFileSizeAuthor
textinput.patch577 bytespifantastic
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kleinmp’s picture

kleinmp CreditAttribution: kleinmp commented

That's a good idea. The building of the list of form ids will have to be done a little differently in order to take advantage of this.

pifantastic’s picture

pifantastic CreditAttribution: pifantastic commented

@kleinmp what do you mean exactly? I'm happy to create another patch if simply changing the field from textfield to textarea isn't enough.

CodigoVision’s picture

CodigoVision CreditAttribution: CodigoVision commented
Issue summary: View changes

Hi, It would be great if this could be added to 7.x also. I manually made the same change and it works fine. Thanks!

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor as a volunteer and at Drupal Ukraine Community for Drupal Ukraine Community commented
Status: Active » Closed (outdated)