Allow control of the h1 page title tag

I suppose all that's really necessary is to turn on the title control for the display, which will give the usual Title choice + selector widget.

Though when rendering we probably then need to react to this setting.

This is just a quick setting in the $form_state in the content editor. Can you give this a try and see if this suits your purpose?

It looks like the panelizer task should automatically react properly since it just returns the data from render_entity() -- but instead of using drupal_set_title() you should put the title in the $info array returned from render_entity() I think.

Compare to panel_context.inc:

  $info = array(
    'content' => panels_render_display($display, $renderer),
    'no_blocks' => !empty($handler->conf['no_blocks']),
  );

  $info['title'] = $display->get_title();

I got an error after applying the patch from #4 in the admin/config/content/panelizer/node/NODE_TYPE/content form

Notice: Undefined property: stdClass::$display_title in panelizer_edit_content_form() (line 393 of /var/www/prod/profiles/sclerosis/modules/contrib/panelizer/includes/common.inc)

For some unknown reason the $cache object dosen't contain property display_title. Display title appear as a property on the object display instead. The form_item to choose how to alter the title dosen't appear if the title isn't set therefore I pass an empty string.

The attached rewrite of the above patch solved my problem, though I don't believe it's the right way to go but maybe useful for others.

I'm using following versions of core and contrib:

• drupal 7.14
• ctools 1.0
• panels 3.2
• panelizer 2.0
• panels_everywhere 1.0-alpha1
• panels_ipe 3.2

+++ b/includes/common.incundefined
@@ -390,7 +390,7 @@ function panelizer_edit_content_form($form, &$form_state) {
-  $form_state['display_title'] = !empty($cache->display_title);
+  $form_state['display_title'] = $cache->display_title;

In the 3.x branch, this is just set to TRUE here. I think this is the correct action.

Also in the 3.x branch, $info['title'] is actually set inside render_entity() -- I believe we should be consistent.

Here is a patch against the 3.x branch.

It sounds more like this is a bug report.

Here's a reroll of #5 taking the feedback from #6 into account for the 2.x branch.

Patch works fine in local testing.

Looks like I committed this a couple weeks ago and forgot to mark it fixed for both branches.

.

@pinkonomy: It already is.

.

@pinkonomy: This are the options you get with Panelizer v3:

If you do not see those settings, perhaps Panopoly is doing something strange. I suspect you might want to start with an issue in the Panopoly issue queue, if it turns out that there is a problem with Panelizer please open a new issue. Thanks.

.

@pinkonomy: Please open an issue with Panolopy.

pinkonomy: make sure you're using full page override.
Also make sure you go to structure > pages and enable the node template for node view, without doing that you will still see the page title.

@dave.erwin
With that solution if you set "No title" is also removed from the causing SEO issues.
Can be solved with this
https://www.drupal.org/node/2262173