[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.




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.







More information about the drupal-devel mailing list