Here's a patch that gives each forum a CSS ID on the top level (www.whatever.com/forum) page. The wrapper div that indents the sub-forums now contains an ID of the format "forum-forum-$tid", similar to the block ids.

This will allow customisation of the forum display. It will allow themes to add forum icons using background-images, which is a popular feature of other forum software, or could be used to highlight a support/FAQ forum in some way.

CommentFileSizeAuthor
forum.module_4.patch1.14 KBixis.dylan

Comments

ixis.dylan’s picture

ixis.dylan commented

To clarify, an example div would be:

<div id="forum-1" style="(indentation)">

The patch changes two identical lines.

Bèr Kessels’s picture

Bèr Kessels commented

are you awaer of the more general approach at http://drupal.org/node/23584 ?

ixis.dylan’s picture

ixis.dylan commented

I'm aware of the discussion, yes. Has anything happened with that since May? It sounds like a great idea, but until that makes it into core (and until somebody rewrites the forum module to take advantage of it), this is a simple patch that adds some interesting possibilities to the forum layout. It won't break anything when/if your wrapper enters core, and it doesn't change any functionality or conflict with anything.

I agree that a theme function would be a better way to do this in the long term, but right now this would be useful for anybody writing a theme who wants to fool about with the forum layout. Like me.

ixis.dylan’s picture

ixis.dylan commented

Now that I think about it (and I'm sober), I should probably just add a theme function to the forum module to do this.

robin monks’s picture

robin monks commented
Status: Needs review » Closed (duplicate)

Making as a duplicate of http://drupal.org/node/23584

Robin
I ♥ Buz