Additional reply classes

[jaredatch]

Right now the default div for a reply looks like:

<div id="post-8" class="post-8 reply type-reply status-publish hentry even bbp-parent-forum-5 bbp-parent-topic-7">

It would be pretty nifty if we could add a few classes if the conditionals are met. Off the top of my head:

• 'topic-author', could be used to highly responses from the topic's original author
• 'forum-admin' / 'forum-moderator' - style admin/moderator replies to stand out
• 'user-%username%' - could be used to style/highly specific users

There might be some other good ones, this is what I came up with on first pass.

As per http://bbpress.org/forums/topic/how-to-change-the-admin-reply-and-user-reply-not-same-color?replies=3#post-98795

[jaredatch]

Patch attached, tacks on the following to reply classes:

user-%username% - currently using str_replace to remove spaces in favor for hyphens, this doesn't seem smart. Maybe just use user ID, eg user-1? Either way they can target specific users with it via CSS.

topic-author - if reply author is the same as original topic author

forum-admin or forum-moderator - think that says it all. Right now it's in an if-statement that looks out of place in the function, but I wasn't sure where else to put it.

See what you think :)

[johnjamesjacoby]

(In [3785]) CSS class changes

• Add user-id and topic-author to reply class
• Add user-id to topic class
• Run reply classes through array_filter()
• Fixes #1763
• Props jaredatch

[johnjamesjacoby]

Checking each user's role and caps ended up to be a performance hit of potentially two queries per post. Not willing to take that hit yet, so only included the ones that were previously cached.

[jaredatch]

Did you use debug bar to test/bench that? I'm curious so I can save you some testing when I submit future patches :P