Ticket #6513: 6513.diff

src/bp-groups/bp-groups-widgets.php

@@ -72 +72 @@
                  */
                 $title = apply_filters( 'widget_title', $instance['title'], $instance, $this->id_base );
 
+                /**
+                 * Filters the separator of the group list options
+                 */
+                $separator = apply_filters( 'bp_groups_widget_separator', '|' );
+
                 echo $before_widget;
 
                 $title = !empty( $instance['link_title'] ) ? '<a href="' . bp_get_groups_directory_permalink() . '">' . $title . '</a>' : $title;
@@ -89 +94 @@
 
                 <?php if ( bp_has_groups( $group_args ) ) : ?>
                         <div class="item-options" id="groups-list-options">
-                                <a href="<?php bp_groups_directory_permalink(); ?>" id="newest-groups"<?php if ( $instance['group_default'] == 'newest' ) : ?> class="selected"<?php endif; ?>><?php _e("Newest", 'buddypress') ?></a> |
-                                <a href="<?php bp_groups_directory_permalink(); ?>" id="recently-active-groups"<?php if ( $instance['group_default'] == 'active' ) : ?> class="selected"<?php endif; ?>><?php _e("Active", 'buddypress') ?></a> |
+                                <a href="<?php bp_groups_directory_permalink(); ?>" id="newest-groups"<?php if ( $instance['group_default'] == 'newest' ) : ?> class="selected"<?php endif; ?>><?php _e("Newest", 'buddypress') ?></a>
+                                <span class="bp-separator" role="separator"><?php echo esc_html( $separator ); ?></span>
+                                <a href="<?php bp_groups_directory_permalink(); ?>" id="recently-active-groups"<?php if ( $instance['group_default'] == 'active' ) : ?> class="selected"<?php endif; ?>><?php _e("Active", 'buddypress') ?></a>
+                                <span class="bp-separator" role="separator"><?php echo esc_html( $separator ); ?></span>
                                 <a href="<?php bp_groups_directory_permalink(); ?>" id="popular-groups" <?php if ( $instance['group_default'] == 'popular' ) : ?> class="selected"<?php endif; ?>><?php _e("Popular", 'buddypress') ?></a>
                         </div>
 

src/bp-members/bp-members-widgets.php

@@ -76 +76 @@
                 $title = apply_filters( 'widget_title', $settings['title'], $settings, $this->id_base );
                 $title = $settings['link_title'] ? '<a href="' . bp_get_members_directory_permalink() . '">' . $title . '</a>' : $title;
 
+                /**
+                 * Filters the separator of the member list options
+                 */
+                $separator = apply_filters( 'bp_members_widget_separator', '|' );
+
                 // Output before widget HTMl, title (and maybe content before & after it)
                 echo $args['before_widget']
                    . $args['before_title']
@@ -96 +101 @@
 
                         <div class="item-options" id="members-list-options">
                                 <a href="<?php bp_members_directory_permalink(); ?>" id="newest-members" <?php if ( 'newest' === $settings['member_default'] ) : ?>class="selected"<?php endif; ?>><?php esc_html_e( 'Newest', 'buddypress' ); ?></a>
-                                |  <a href="<?php bp_members_directory_permalink(); ?>" id="recently-active-members" <?php if ( 'active' === $settings['member_default'] ) : ?>class="selected"<?php endif; ?>><?php esc_html_e( 'Active', 'buddypress' ); ?></a>
+                                <span class="bp-separator" role="separator"><?php echo esc_html( $separator ); ?></span>
+                                <a href="<?php bp_members_directory_permalink(); ?>" id="recently-active-members" <?php if ( 'active' === $settings['member_default'] ) : ?>class="selected"<?php endif; ?>><?php esc_html_e( 'Active', 'buddypress' ); ?></a>
 
                                 <?php if ( bp_is_active( 'friends' ) ) : ?>
+                                        <span class="bp-separator" role="separator"><?php echo esc_html( $separator ); ?></span>
+                                        <a href="<?php bp_members_directory_permalink(); ?>" id="popular-members" <?php if ( 'popular' === $settings['member_default'] ) : ?>class="selected"<?php endif; ?>><?php esc_html_e( 'Popular', 'buddypress' ); ?></a>
 
-                                        | <a href="<?php bp_members_directory_permalink(); ?>" id="popular-members" <?php if ( 'popular' === $settings['member_default'] ) : ?>class="selected"<?php endif; ?>><?php esc_html_e( 'Popular', 'buddypress' ); ?></a>
-
                                 <?php endif; ?>
 
                         </div>