The field that holds the path when you set a pane title to be a link is set to 255 characters, but the maxlength of the form field in the UI is 128 (the default for text fields). If the field can hold 255 characters, it probably makes sense to allow the UI to accept 255 characters.

CommentFileSizeAuthor
#2 3064054-fieldable_panels_panes-title-link-length-2.patch404 bytescboyden

Comments

cboyden created an issue. See original summary.

cboyden’s picture

cboyden commented
Status: Active » Needs review
StatusFileSize
new 3064054-fieldable_panels_panes-title-link-length-2.patch404 bytes

Patch is attached.

Also it looks like this might be an issue for more than just this field. If maintainers think it's a good idea I can make this change for other text fields in the UI where appropriate.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Assigned: cboyden » Unassigned
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #3039440: Plan for Fieldable Panels Panes 7.x-1.14
michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Needs review » Reviewed & tested by the community

I don't know what other text fields it's needed on but it works great for this one. Thanks!

  • DamienMcKenna committed cc58810 on 7.x-1.x authored by cboyden 
    Issue #3064054 by cboyden, DamienMcKenna, Michelle: Link path field cuts...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Committed. Thank you!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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