in admin/appearance/fontyourface where I can add my custom css classes the form-type-textfield has max length of 128.
would be great to change this...

thx for the great module!

Comments

whthat’s picture

Version: 7.x-2.x-dev » 7.x-2.0
Component: Miscellaneous » User interface
Priority: Minor » Normal

This is also the case for the 7.x-2.0 version. Although you can browse for the font to get to the full settings page and add more than 128 characters. This takes a bit of extra time to complete. Another option may be just allowing access to the full font settings page from the main Font-Your-Face view.

sreynen’s picture

Version: 7.x-2.0 » 7.x-2.x-dev

This will need to be fixed in the dev version first. This should be a very simple fix, a great opportunity for a new contributor to get some experience making a patch. It's also a pretty rare use case and there's already a work-around, so I'm going to wait a bit on this in hopes that it can be a learning experience for someone.

whthat’s picture

I made a patch to fix the maxlength, the 128 value seems to be an inherited value. I hard coded a maxlength of 1024 into this fix. Hope the patch is formatted correctly (Git first timer).

sreynen’s picture

Looks good, thanks. I'll try to get it committed soon.

sreynen’s picture

Status: Active » Needs review

I'm going to change this to "needs review" just to see what the testbot says, more to make sure the tests still work than to test the patch.

sreynen’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.