Changeset 6878 for trunk/bp-groups/bp-groups-widgets.php

Timestamp:

04/01/2013 10:35:40 PM (12 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:

• trunk/bp-groups/bp-groups-widgets.php (modified) (1 diff)

trunk/bp-groups/bp-groups-widgets.php

@@ -165 +165 @@
     if ( bp_has_groups( 'type=' . $type . '&per_page=' . $_POST['max_groups'] . '&max=' . $_POST['max_groups'] ) ) : ?>
         <?php echo "0[[SPLIT]]"; ?>
-
-        <ul id="groups-list" class="item-list">
-            <?php while ( bp_groups() ) : bp_the_group(); ?>
-                <li>
-                    <div class="item-avatar">
-                        <a href="<?php bp_group_permalink() ?>"><?php bp_group_avatar_thumb() ?></a>
+        <?php while ( bp_groups() ) : bp_the_group(); ?>
+            <li>
+                <div class="item-avatar">
+                    <a href="<?php bp_group_permalink() ?>"><?php bp_group_avatar_thumb() ?></a>
+                </div>
+
+                <div class="item">
+                    <div class="item-title"><a href="<?php bp_group_permalink() ?>" title="<?php bp_group_name() ?>"><?php bp_group_name() ?></a></div>
+                    <div class="item-meta">
+                        <span class="activity">
+                            <?php
+                            if ( 'newest-groups' == $_POST['filter'] ) {
+                                printf( __( 'created %s', 'buddypress' ), bp_get_group_date_created() );
+                            } else if ( 'recently-active-groups' == $_POST['filter'] ) {
+                                printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() );
+                            } else if ( 'popular-groups' == $_POST['filter'] ) {
+                                bp_group_member_count();
+                            }
+                            ?>
+                        </span>
                     </div>
-
-                    <div class="item">
-                        <div class="item-title"><a href="<?php bp_group_permalink() ?>" title="<?php bp_group_name() ?>"><?php bp_group_name() ?></a></div>
-                        <div class="item-meta">
-                            <span class="activity">
-                                <?php
-                                if ( 'newest-groups' == $_POST['filter'] ) {
-                                    printf( __( 'created %s', 'buddypress' ), bp_get_group_date_created() );
-                                } else if ( 'recently-active-groups' == $_POST['filter'] ) {
-                                    printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() );
-                                } else if ( 'popular-groups' == $_POST['filter'] ) {
-                                    bp_group_member_count();
-                                }
-                                ?>
-                            </span>
-                        </div>
-                    </div>
-                </li>
-
-            <?php endwhile; ?>
-        </ul>
+                </div>
+            </li>
+        <?php endwhile; ?>
+
         <?php wp_nonce_field( 'groups_widget_groups_list', '_wpnonce-groups' ); ?>
         <input type="hidden" name="groups_widget_max" id="groups_widget_max" value="<?php echo esc_attr( $_POST['max_groups'] ); ?>" />