Love this module! However, the one thing I'd like to see is the title and content sections wrapped in div with a unique class identifier. That would make theming so much easier.
As an example, I have 5 different content type set as content profile. Since the the content_profile-display-view.tpl.php has the title and content as separate divs with no unique identifiers (the only unique is called by $content inside the content-profile-display div), I can't easily take each different type and style it in css (move it around as a whole) without affecting all the types.

Comments

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Active » Fixed

I've added unique id's to the div and the h3 by default -> committed to 6.x-1.x-dev.

fred0’s picture

fred0 commented

Ah, yes, that helps, but what I meant was a wrapper for both title and content like this:

<?php
// $Id: content_profile-display-view.tpl.php,v 1.1.2.2 2009/01/04 11:46:29 fago Exp $

/**
 * @file content-profile-display-view.tpl.php
 *
 * Theme implementation to display a content-profile.
 */
?>
<div id="content-profile-<?php print $type; ?>">
<?php if (isset($title)): ?>
  <h3 class="content-profile-title" id="content-profile-title-<?php print $type; ?>">
  <?php print $title; ?>
  </h3>
<?php endif; ?>
<div class="content-profile-display" id="content-profile-display-<?php print $type; ?>">
  <?php if (isset($tabs)) : ?>
    <ul class="tabs content-profile">
      <?php foreach ($tabs as $tab) : ?>
        <?php if ($tab): ?>
          <li><?php print $tab; ?></li>
        <?php endif; ?>
      <?php endforeach; ?>
    </ul>
  <?php endif; ?>
  <?php if (isset($node->nid) && isset($content)): ?>
    <?php print $content ?>
  <?php endif; ?>
</div>
</div>

Easy enough tweak for me as a template override in my theme folder, but your call if you want to commit it.
Thanks!

Status: Fixed » Closed (fixed)

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