Theme forum post preview to match forum theme

blogjunkie - January 10, 2008 - 15:31
Project:Advanced Forum
Version:5.x-1.x-dev
Component:Theming
Category:task
Priority:normal
Assigned:Michelle
Status:closed
Description

Hi Michelle,

First of all, thanks so much for this awesome awesome module.

Second, I wanted to ask if there's a way to get the forum post preview to look like the advanced forum theme? At the moment when my site users click preview they see the regular comment preview so its a slight bump in the user experience. Appreciate any help you can give me.

Thanks!

#1

Michelle - January 16, 2008 - 04:22

This is one of the things I need to figure out how to do. Leaving this active to remind me to do it. :)

Michelle

#2

Michelle - January 24, 2008 - 20:45
Title:Forum post preview shows old theme?» Theme forum post preview to match forum theme
Category:support request» task
Assigned to:Anonymous» Michelle

#3

neclimdul - June 2, 2008 - 05:38
Status:active» needs review

Possible fix. It assumes that a site wants all output of forum posts that aren't teasers to look like a forum post. This isn't necessarily true as someone could have a full node listing view.

I think its the best general solution though as it will preform well and most cases where this would be a problem should be able to work around this.

#4

neclimdul - June 2, 2008 - 05:39

patch.

Am I really this rusty at posting patches or should I stop trying to post patches after 12?

AttachmentSize
208440.patch 827 bytes

#5

Michelle - June 3, 2008 - 19:30
Status:needs review» fixed

@neclimdul - Thanks, but I don't care for this solution. The $teaser variable is unreliable and, as you said, people may have views with full nodes elsewhere. Since I was looking at it anyway, I went ahead and got it working by adding "reply" to the URL list. I also removed the links from the comment since they looked silly there. I tried to remove them from the node but it's being stubborn so I'll save that for another time.

Thanks,

Michelle

#6

Anonymous (not verified) - June 17, 2008 - 19:34
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.