[Revan]

I've made the following changes to this mod in order to make it fit into my forums more cleanly, and it'd be nice if some of these changes made it into the final mod (maybe via options)?

Variable: Placement changed:

Under $forumbits in the FORUMHOME template.

Change/Reasoning:

• Changed placement to make it snap to the bottom of the forum list, in order to make it more in line with the rest of the forum lists.

Template forumhome_cel_social_groups:

<if condition="$show_cel_sgfh">
<if condition="($cel_groups_hasresults) OR ($vboptions[cel_sgfh_show_teaser])">
<tbody>
    <tr>
    <td class="tcat" colspan="5">
        <a style="float:$stylevar[right]" href="#top" onclick="return toggle_collapse('new_social_disc');"><img id="collapseimg_new_social_disc" src="$stylevar[imgdir_button]/collapse_tcat$vbcollapse[collapseimg_new_social_disc].gif" alt="" border="0" /></a>
        <a href="group.php$session[sessionurl_q]">$vbphrase[social_groups]</a>$vbphrase[cel_sgfh_forumhome_title]
    </td>
    </tr>
</tbody>
    <tbody id="collapseobj_new_social_disc" style="$vbcollapse[collapseobj_new_social_disc]">
        $cel_sgfh_output
    </tbody>
    <if condition="$cel_groups_limit">
<tbody>
    <tr>
    <td class="tfoot smallfont" colspan="5" align="center">
        <strong>
            <a href="$cel_groups_show_more_link_href">$vbphrase[cel_sgfh_show_more]</a>
        </strong>
    </td>
    </tr>
</tbody>
    </if>
</if>
</if>

Changes/Reasoning:

• Removed the custom header in order to make it respect the width of the forumbits.

Template forumhome_cel_social_groups_groupsbit:

<if condition="($cel_groups_hasresults)">
<tr>
<td class="alt2Active">
    <img src="$stylevar[imgdir_statusicon]/$forum[imageprefix]$statusimg.gif" alt="" border="0" />
</td>
<td class="alt1">
    <if condition="$vboptions[cel_sgfh_showbuttons]">
        <div style="float: right; padding: 3px;">
            <a href="group.php?$session[sessionurl]do=message&amp;groupid=$cel_groups[groupid]"><img src="$stylevar[imgdir_button]/cel_sgfh_new_discussion.png" alt="$vbphrase[cel_sgfh_new_discussion]" border="0" /></a>
            <a href="group.php?$session[sessionurl]do=viewmembers&amp;groupid=$cel_groups[groupid]"><img src="$stylevar[imgdir_button]/cel_sgfh_members.png" alt="$vbphrase[cel_sgfh_members_button]" border="0" /></a>
            <if condition="$cel_groups_pictures"><a href="group.php?$session[sessionurl]do=grouppictures&amp;groupid=$cel_groups[groupid]"><img src="$stylevar[imgdir_button]/cel_sgfh_picture.png" alt="$vbphrase[cel_sgfh_picture]" border="0" /></a></if>
            <if condition="$vboptions[cel_sgfh_show]==subscribed"><a href="group.php?$session[sessionurl]do=unsubscribe&amp;groupid=$cel_groups[groupid]"><img src="$stylevar[imgdir_button]/cel_sgfh_unsubscribe.png" alt="$vbphrase[cel_sgfh_unsubscribe]" border="0" /></a></if>
        </div>
    </if>
    <a href="group.php?$session[sessionurl]groupid=$cel_groups[groupid]">$cel_groups[name]</a><br /><span class="smallfont">$cel_groups[description]</span>
</td>
<td class="alt2 smallfont">
    <div style="white-space: nowrap">
        $cel_sgfh_output_lastmsg
    </div>
</td>
<td class="alt1" align="center">
    $cel_groups[discussions]
</td>
<td class="alt2" align="center">
    $cel_groups[visible]
</td>
</tr>

<else />
    <tr>
        <td class="alt1" colspan="5">$cel_groups_no_result_phrase</td>
    </tr>
</if>

Changes/Reasoning:

• Removed hard-coded .gif extension from the plugin in order to support skins using transparent PNGs as their forum_on and forum_off images.
• Changed members to discussions (equivalent of threads).

Template forumhome_cel_social_groups_lastmsgby:

<div class="smallfont" align="$stylevar[left]">
    <div>
        <span style="white-space:nowrap">
        <a href="group.php?$session[sessionurl]do=discuss&amp;gmid=$cel_groups[lastgmid]" style="white-space:nowrap" title="$vbphrase[go_to_last_post]"><strong>$cel_groups[trimmed_lastdiscussion]</strong></a></span>
    </div>
    <div style="white-space:nowrap<if condition="is_browser('ie', 6)">; float:$stylevar[left]</if>">
        <phrase 1="member.php?$session[sessionurl]u=$cel_groups[lastposterid]" 2="$cel_groups[lastposter]">$vbphrase[by_x]</phrase>
    </div>
    <div align="$stylevar[right]" style="white-space:nowrap">
        $cel_groups[lastmsgdate] <if condition="!$show['detailedtime']"><span class="time">$cel_groups[lastmsgtime]</span></if>
        <a href="group.php?$session[sessionurl]do=discuss&amp;gmid=$cel_groups[lastgmid]#gmessage$cel_groups[lastgmid]"><img class="inlineimg" src="$stylevar[imgdir_button]/lastpost.gif" alt="$vbphrase[go_to_last_post]" border="0" /></a>
    </div>
</div>

Changes/Reasoning:

• Fixed &gmid to &gmid, because of XHTML validity.

Plugin forumhome code:

                if ($cel_groups['readtime'] AND $cel_groups['lastpost'] > $cel_groups['readtime'])
                {
                    $statusimg = "forum_new";
                }
                else
                {
                    $statusimg = "forum_old";
                }

Changes/Reasoning:

• Removed hard-coded .gif extension from the plugin in order to support skins using transparent PNGs as their forum_on and forum_off images.

I believe these changes, especially if added as options, will be beneficial for those like me, who prefer to have the Social Groups list exactly the same way as any other forum category