Issue status update for http://drupal.org/node/22487 Post a follow up: http://drupal.org/project/comments/add/22487 Project: Drupal Version: cvs Component: theme system Category: feature requests Priority: normal Assigned to: willmoy Reported by: willmoy Updated by: willmoy -Status: patch (code needs review) +Status: patch (ready to be committed) Ready to be committed if accepted. willmoy Previous comments: ------------------------------------------------------------------------ Tue, 10 May 2005 01:36:10 +0000 : 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. ------------------------------------------------------------------------ Tue, 10 May 2005 11:30:24 +0000 : 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. ------------------------------------------------------------------------ Tue, 10 May 2005 14:42:04 +0000 : stefan nagtegaal I do think this is nicer than the way of having multiple node.$type.tpl.php files as we have now... ------------------------------------------------------------------------ Tue, 10 May 2005 14:49:33 +0000 : 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. ------------------------------------------------------------------------ Wed, 11 May 2005 18:33:36 +0000 : factoryjoe This will be in the upcoming versions of SpreadSpread (was SpreadFirefox), Lincoln's Revenge and Democratica.