Fieldable Panels Panes is lacking the option to link the title of a pane to another page or URL, like you can when you place a node or a view in a panel. I guess the difference is nodes and views already have paths.

Attached patch adds a link checkbox and a path textfield and makes the title in to a link in fieldable_panels_panes_preprocess_panels_pane(). There is no validation or filtering but I believe that should be safe as it goes through the l() function.

This seemed to be the simplest approach, but other thoughts I had were to use a link or node reference field in the fieldable entity and use that to preprocess the title, or alternatively provide this functionality in Panels, but I'm not sure how to go about that considering some plugins already provide linking ability.

Comments

mstrelan’s picture

Slight improvement to the weight of the link field.

mstrelan’s picture

And I guess it makes sense to check the title is not blank first, so we don't print an empty link.

mstrelan’s picture

So sorry for the noise, but I messed up the description too, by copy/pasting from the edit menu link UI , rather than from the menu module. This is the last one for now, I promise.

cangeceiro’s picture

Status: Needs review » Fixed

makes sense, I changed the weight so the settings always reside with the title. committed. Thanks.

Status: Fixed » Closed (fixed)

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