It appears that there are several issues concerning the pane title when one opts to use panelizer to control the display of a fieldable panel pane bundle.

To start, setting the title setting to 'no title' is not respected (still outputs the pane title with standard formatting).

Secondly, using the 'title' field content type does not work either.

Thus, what I'm getting with these settings:

Title:No title

field:title
field:other_field

I'm getting:

Title

Other field

I hope that makes sense? It's very different from the behavior that I'm getting using panelizer with other entity types. I'm using semantic panels, but I cannot define a default semantic panel style for a specific FPP bundle (wouldn't that be nice?) thus I'm looking to panelize the default display to achieve this effect, so one does not have to style each individual fpp pane as its created.

Files: 
CommentFileSizeAuthor
#9 2015383-panelizer-title-lost-8.patch1.24 KBfubhy
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]
#7 2015383-panelizer-title-lost.patch1.2 KBfubhy
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 2015383-panelizer-title-lost_0.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#2 2015383-panelizer-title-lost.patch1.24 KBmerlinofchaos
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Comments

It also might be worth mentioning that I've applied #1588882: Use title for entity label and make required

Status:Active» Needs review
StatusFileSize
new1.24 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

This patch is completely untested and might need some tweaks, but it should be close to fixing this.

Hmm...no dice... I'll try this in a sandbox and see if I can't isolate it. I'm wondering if the fact that I'm using panopoly has anything to do with it. The code certainly looks like it should work!

Ok, tested it in a vanilla install, still not working. Using Panelizer, FPP, Panels, Ctools, etc all --dev from drush. No panoply this time (panopoly_magic messes with FPP titles a bit I think?)

It looks like both load and render are doing their job from the devel end of things.

hide_title is set to 1, and the content wants to show fieldable_panels_pane:title, however the panelized content is rendering blank, and the pane title is still showing in the default way.

heres the #panelizer_content array if you are interested:

#panelizer_content (Array, 4 elements)
content (String, 437 characters )

<div class="panel-flexible panels-flexible-2 clearfix" >
<div class="panel-flexible-inside panels-flexible-2-inside">
<div class="panels-flexible-region panels-flexible-region-2-center panels-flexible-region-first panels-flexible-region-last">
  <div class="inside panels-flexible-region-inside panels-flexible-region-2-center-inside panels-flexible-region-inside-first panels-flexible-region-inside-last">
  </div>
</div>
</div>
</div>

no_blocks (Boolean) FALSE
classes_array (Array, 0 elements)
title (String, 0 characters )

There are a few other FPP<>Panelizer bugs that I will report in other issues! Let me know how else I can help, thanks a bunch!

Oh, the code is checking !empty() on the title. Change that to isset().

I tried fiddling around with the patch - it seems that it will make titles dissapear properly if they are set to no title, however the extra_field fieldable_panels_pane:title still isn't displaying in the content at all when the panelizer default template is set to no title. I'll see if I can't extend this patch to make it do that as well.

StatusFileSize
new1.2 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 2015383-panelizer-title-lost_0.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Re-uploading fixed patch. I can confirm that this works with isset() now.

Status:Needs review» Needs work

The last submitted patch, 2015383-panelizer-title-lost.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new1.24 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Sorry, screwed up the patch

The patch in #9 works for me. Thank you.

Issue summary:View changes
Status:Needs review» Reviewed & tested by the community

RTBC then. ;)

Just tested this patch as well. Makes FPP and Panelizer work as expected now. Thanks!