Skip to:
Content

BuddyPress.org

Changeset 6878


Ignore:
Timestamp:
04/01/2013 10:35:40 PM (11 years ago)
Author:
djpaul
Message:

Don't wrap the groups widget AJAX response in a <UL> element. Fixes #4903

This tag is already present in the surrounding area of the template, so this avoids duplicate markup.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/bp-groups/bp-groups-widgets.php

    r6865 r6878  
    165165    if ( bp_has_groups( 'type=' . $type . '&per_page=' . $_POST['max_groups'] . '&max=' . $_POST['max_groups'] ) ) : ?>
    166166        <?php echo "0[[SPLIT]]"; ?>
    167 
    168         <ul id="groups-list" class="item-list">
    169             <?php while ( bp_groups() ) : bp_the_group(); ?>
    170                 <li>
    171                     <div class="item-avatar">
    172                         <a href="<?php bp_group_permalink() ?>"><?php bp_group_avatar_thumb() ?></a>
     167        <?php while ( bp_groups() ) : bp_the_group(); ?>
     168            <li>
     169                <div class="item-avatar">
     170                    <a href="<?php bp_group_permalink() ?>"><?php bp_group_avatar_thumb() ?></a>
     171                </div>
     172
     173                <div class="item">
     174                    <div class="item-title"><a href="<?php bp_group_permalink() ?>" title="<?php bp_group_name() ?>"><?php bp_group_name() ?></a></div>
     175                    <div class="item-meta">
     176                        <span class="activity">
     177                            <?php
     178                            if ( 'newest-groups' == $_POST['filter'] ) {
     179                                printf( __( 'created %s', 'buddypress' ), bp_get_group_date_created() );
     180                            } else if ( 'recently-active-groups' == $_POST['filter'] ) {
     181                                printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() );
     182                            } else if ( 'popular-groups' == $_POST['filter'] ) {
     183                                bp_group_member_count();
     184                            }
     185                            ?>
     186                        </span>
    173187                    </div>
    174 
    175                     <div class="item">
    176                         <div class="item-title"><a href="<?php bp_group_permalink() ?>" title="<?php bp_group_name() ?>"><?php bp_group_name() ?></a></div>
    177                         <div class="item-meta">
    178                             <span class="activity">
    179                                 <?php
    180                                 if ( 'newest-groups' == $_POST['filter'] ) {
    181                                     printf( __( 'created %s', 'buddypress' ), bp_get_group_date_created() );
    182                                 } else if ( 'recently-active-groups' == $_POST['filter'] ) {
    183                                     printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() );
    184                                 } else if ( 'popular-groups' == $_POST['filter'] ) {
    185                                     bp_group_member_count();
    186                                 }
    187                                 ?>
    188                             </span>
    189                         </div>
    190                     </div>
    191                 </li>
    192 
    193             <?php endwhile; ?>
    194         </ul>
     188                </div>
     189            </li>
     190        <?php endwhile; ?>
     191
    195192        <?php wp_nonce_field( 'groups_widget_groups_list', '_wpnonce-groups' ); ?>
    196193        <input type="hidden" name="groups_widget_max" id="groups_widget_max" value="<?php echo esc_attr( $_POST['max_groups'] ); ?>" />
Note: See TracChangeset for help on using the changeset viewer.