Changeset 9955 for trunk/src/bp-groups/bp-groups-widgets.php

Timestamp:

06/20/2015 01:23:47 PM (10 years ago)

Author:

boonebgorges

Message:

Improve the flexibility of the separators in the action links of the groups and members widgets.

• Introduce filters 'bp_groups_widget_separator' and 'bp_members_widget_separator' to allow customization of the default pipe character.
• Wrap separators in a <span> element, to allow for custom styling and proper ARIA role.

Props dunhakdis, pareshradadiya, hnla.
Fixes #6513.

File:

• trunk/src/bp-groups/bp-groups-widgets.php (modified) (2 diffs)

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

@@ -73 +73 @@
         $title = apply_filters( 'widget_title', $instance['title'], $instance, $this->id_base );
 
+        /**
+         * Filters the separator of the group widget links.
+         *
+         * @since BuddyPress (2.4.0)
+         *
+         * @param string $separator Separator string. Default '|'.
+         */
+        $separator = apply_filters( 'bp_groups_widget_separator', '|' );
+
         echo $before_widget;
 
@@ -90 +99 @@
         <?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>