Ensure $title_suffix is always rendered (i.e. fix contextual links)

the patch does not fix it for me. in my opinion this is a bug not a feature request.

Here's the workaround I'm using for now to solve this issue. The code is largely taken from Views's views_preprocess_html function. The problem is that the class that Views looks for in that function (views_contextual_links_info) is not in the HTML template in Omega; it's in the Page template. Ultimately, it would good to fix that in Omega, if it doesn't break something else. In the meantime, this theme preprocess function should work. It should go either in preprocess/preprocess-page.inc or in template.php.

function THEMENAME_alpha_preprocess_page(&$vars) {
  if (!empty($vars['page']['#views_contextual_links_info'])) {
    $key = array_search('contextual-links-region', $vars['attributes_array']['class']);
    if ($key !== FALSE) {
      unset($vars['attributes_array']['class'][$key]);

      // Add the JavaScript, with a group and weight such that it will run
      // before modules/contextual/contextual.js.
      drupal_add_js(drupal_get_path('module', 'views') . '/js/views-contextual.js', array('group' => JS_LIBRARY, 'weight' => -1));
    }
  }
}

The actual solution for this is quite simple via a quick template edit.

<div<?php print $attributes; ?>>
  <div<?php print $content_attributes; ?>>
    <a id="main-content"></a>
    <?php print render($title_prefix); ?>
    <?php if ($title): ?>
    <?php if ($title_hidden): ?><div class="element-invisible"><?php endif; ?>
    <h1 class="title" id="page-title"><?php print $title; ?></h1>
    <?php if ($title_hidden): ?></div><?php endif; ?>
    <?php endif; ?>
    <?php print render($title_suffix); ?>
    <?php if ($tabs && !empty($tabs['#primary'])): ?><div class="tabs clearfix"><?php print render($tabs); ?></div><?php endif; ?>
    <?php if ($action_links): ?><ul class="action-links"><?php print render($action_links); ?></ul><?php endif; ?>
    <?php print $content; ?>
    <?php if ($feed_icons): ?><div class="feed-icon clearfix"><?php print $feed_icons; ?></div><?php endif; ?>
  </div>
</div>

The <?php print render($title_prefix); ?> and <?php print render($title_suffix); ?> should appear outside of the conditional <?php if ($title): ?>

This fix is going in the 3.1 release

Forgot to mention that this template edit is in region--content.tpl.php for page level title rendering, and also in node.tpl.php for default teaser type title rendering.

I submitted this patch #1270552: $title_prefix, $title_suffix not outside "if ($title)" in region--content.tpl.php (just for region--content.tpl.php), but it fell between the cracks, and today got marked as duplicate even though it preceded this issue by over a month. Also, when marking as duplicate, it's helpful to indicate the issue it is a duplicate for.

Yeah this doesn't work for me either on node teasers. The title_prefix element is empty. The title_suffix element contains the contextual links but the links are not appearing.

Scratch that. It does work. It just doesn't work with Contextual Flyout Links. Bummer.

For anyone interested in making Contextual Flyout Links work for all regions in Omega, try my patch.

The issue that the whole page of a view becomes a contextual links region, resulting in all contextual links being triggered at once is still not solved. I posted it here: http://drupal.org/node/1326744, but it was recommended to continue in this thread

fubhy - any thoughts as to why this would be happening?

Here is the code I'm using to address the Views contextual links issue. It's a shorter version of what I posted above, without the drupal_add_js, since the necessary JS is already added by Views. It's not a perfect solution, but it removes the contextual link trigger from the whole page. The code should go in either THEMENAME/preprocess/preprocess-page.inc or THEMENAME/template.php.

function test_alpha_preprocess_page(&$vars) {
  if (!empty($vars['page']['#views_contextual_links_info'])) {
    $key = array_search('contextual-links-region', $vars['attributes_array']['class']);
    if ($key !== FALSE) {
      unset($vars['attributes_array']['class'][$key]);
    }
  }
}

"test" is the name of my theme, in the code above. Of course, change it to whatever your theme name is.

EDIT: Well, turns out the core module "Contextual Links" got deactivated along the way of creating and developing a subtheme. Re-enabling shows all the contextual links again and I can confirm that 3.1 (and 3.1+4-dev) has all contextual links in the right spot.

EDIT 2: Now that contextual links are present again, I experience the issue with triggering all contextual menus by entering viewpoint. Patch from #17 resolves that issue. When hovering main content on view pages, contextual menus are triggered for both views and all nodes but this also happens with Bartik; so it's not Omega related.

Cheers,
Paul

Hey there,
so I ran into a similar bug. There are no contextual links displayed at all, neither for node pages nor for views.
While using 7.x-3.1, $vars['page'] contains correct '#contextual_links' as well as '#views_contextual_links_info'; however, both $vars['title_prefix'] and $vars['title_suffix'] are empty arrays.
The HTML ouput contains no divs for title's suffix/prefix. I tried all provided patches as in #2, #5 and #17 to no luck.
I tried Omega 7.x-3.1+4-dev (currently latest dev) but again no change.
What am I missing? On which level within $vars should '#contextual_links' and '#views_contextual_links_info' reside to be rendered or, respectively, at which point are these attached to title_suffix?
Thanks in advance,
Paul

Should this be sent to Views as an issue or added to 3.2 in the template.php?

Tried and tested fixes to all but fail.

Resorted to nasty hack as per below:

contexualfix = function() {
	if (jQuery('#page').hasClass('contextual-links-region')) {
        	jQuery('#page').removeClass('contextual-links-region');
	}	
}

Hey guys, somebody founds a definitive solution for this boring issue (without jquery)?

Wow, #2 works perfectly for me! Thanks a lot ;)

The patch in this Views issue worked for me: Contextual links added to page element if 'page' view is displayed

7.x-1.x is unsupported, closing.