[drupal-devel] [feature] Make $node->type a class in the main div of the default node.tpl.php

factoryjoe drupal-devel at drupal.org
Wed May 11 18:33:38 UTC 2005

Issue status update for http://drupal.org/node/22487

 Project:      Drupal
 Version:      cvs
 Component:    theme system
 Category:     feature requests
 Priority:     normal
 Assigned to:  willmoy
 Reported by:  willmoy
 Updated by:   factoryjoe
 Status:       patch

This will be in the upcoming versions of SpreadSpread (was
SpreadFirefox), Lincoln's Revenge and Democratica.


Previous comments:

May 10, 2005 - 01:36 : willmoy

Attachment: http://drupal.org/files/issues/nodetype_theming.patch (716 bytes)

This patch would change the main div of, for example, a sticky book node
<div class="node sticky">
<div class="node node-book sticky">

This is to aid theming; motivated by trying to theme forums differently
from pages etc.

It uses the same convention as node type-specific blocks (e.g.
class="block block-book") and it is should be completely backward
compatible so long as people do not have very odd tastes in class


May 10, 2005 - 11:30 : adrian

You could also just copy node.tpl.php to your theme dir and modify it
there to user $node->type, and if you only want to do something for
forum nodes, you could copy the node.tpl.php to node-forum.tpl.php

PHPTemplate has been moved to core for 4.7, so I will only accept this
patch for the 4.6 branch if it is accepted into core. Otherwise your
templates will just magically stop working.


May 10, 2005 - 14:42 : stefan nagtegaal

I do think this is nicer than the way of having multiple
node.$type.tpl.php files as we have now...


May 10, 2005 - 14:49 : willmoy

Both are useful. This exposes options to CSS that wouldn't otherwise be
there, but there are of course PHP things that you can only do in

This patch augments current functionality, it doesn't replace it or
make it redundant.

More information about the drupal-devel mailing list