[buddypress-trac] [BuddyPress] #4869: BP Theme Compat & BuddyPress Widgets styles

buddypress-trac noreply at wordpress.org
Mon Apr 8 18:10:03 UTC 2013


#4869: BP Theme Compat & BuddyPress Widgets styles
------------------------------------+-----------------------
 Reporter:  imath                   |       Owner:
     Type:  enhancement             |      Status:  reopened
 Priority:  normal                  |   Milestone:  1.8
Component:  Theme                   |     Version:  1.7
 Severity:  normal                  |  Resolution:
 Keywords:  dev-feedback has-patch  |
------------------------------------+-----------------------

Comment (by r-a-y):

 > If we add .widget to an inner div, it could cause all sorts of other
 issues if the .widget class appears on parent elements (as it will in some
 themes).

 True; this was something I was thinking about myself after I posted
 03.patch.

 03b.patch is an alternative patch and should satisfy all requirements.

 * I've renamed all widget classes to `widget-buddypress`
 * I've changed all widget JS to also look for `widget-buddypress` so we
 can keep compatibility with bp-default themes
 * I've changed bp-legacy's CSS from `.widget.buddypress` to `.widget-
 buddypress`

 Tested on bp-default, TwentyTen and TwentyTwelve.

 Let me know what you think.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4869#comment:24>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list