Issue status update for http://drupal.org/node/27776 Post a follow up: http://drupal.org/project/comments/add/27776 Project: Drupal Version: cvs Component: forum.module Category: feature requests Priority: normal Assigned to: leafish_dylan Reported by: leafish_dylan Updated by: Bèr Kessels Status: patch are you awaer of the more general approach at http://drupal.org/node/23584 ? Bèr Kessels Previous comments: ------------------------------------------------------------------------ Fri, 29 Jul 2005 15:40:48 +0000 : leafish_dylan Attachment: http://drupal.org/files/issues/forum.module_4.patch (1.14 KB) 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. ------------------------------------------------------------------------ Fri, 29 Jul 2005 15:44:50 +0000 : leafish_dylan To clarify, an example div would be: <div id="forum-1" style="(indentation)"> The patch changes two identical lines.

Issue status update for http://drupal.org/node/27776 Post a follow up: http://drupal.org/project/comments/add/27776 Project: Drupal Version: cvs Component: forum.module Category: feature requests Priority: normal Assigned to: leafish_dylan Reported by: leafish_dylan Updated by: leafish_dylan Status: patch 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. leafish_dylan Previous comments: ------------------------------------------------------------------------ Fri, 29 Jul 2005 15:40:48 +0000 : leafish_dylan Attachment: http://drupal.org/files/issues/forum.module_4.patch (1.14 KB) 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. ------------------------------------------------------------------------ Fri, 29 Jul 2005 15:44:50 +0000 : leafish_dylan To clarify, an example div would be: <div id="forum-1" style="(indentation)"> The patch changes two identical lines. ------------------------------------------------------------------------ Fri, 29 Jul 2005 16:08:37 +0000 : Bèr Kessels are you awaer of the more general approach at http://drupal.org/node/23584 ? ------------------------------------------------------------------------ Fri, 29 Jul 2005 23:27:52 +0000 : leafish_dylan 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.