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. factoryjoe 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 from: <div class="node sticky"> to: <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 names. ------------------------------------------------------------------------ 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 node-whatever.tpl.php. This patch augments current functionality, it doesn't replace it or make it redundant.