[buddypress-trac] [BuddyPress Trac] #5817: Additional styling support for BuddyPress Widgets

[buddypress-trac]

#5817: Additional styling support for BuddyPress Widgets
----------------------------------------+------------------
 Reporter:  mercime                     |       Owner:
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  2.2
Component:  Theme / Template Parts      |     Version:
 Severity:  normal                      |  Resolution:
 Keywords:  needs-patch good-first-bug  |
----------------------------------------+------------------

Comment (by mercime):

 Thank you DJPaul and r-a-y for your feedback :)

 The patch which fixes missing classes needs to address:
 * if there is no 'class' attribute in the widget container, then we need
 to insert 'class="widget buddypress"' via $before_widget
 * if there is a 'class' attribute but we're not sure if they even added
 "widget" to the class and/or the required variable  %2$s" to output
 "widget buddypress" classes, then we still need to add the classes "widget
 buddypress" via $before_widget to ensure that we have those classes in the
 widget container.
 Working on a patch.

 In the meantime, we also need to address the styles of the BP lists in
 widgets even after we add the necessary classes to the widget. Attached
 are two CSS patches for consideration:
 * 5817-margin-liststyle.patch - addresses only the margins and list styles
 added by some themes
 * 5817-margin-liststyle-avatars.patch - addresses margins, list styles,
 and makes all avatars in BP widgets the same size: 40px x 40px. Screenshot
 with style fixes on this patch is also attached.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5817#comment:8>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac