[buddypress-trac] [BuddyPress Trac] #5817: Additional styling support for BuddyPress Widgets
buddypress-trac
noreply at wordpress.org
Sat Aug 16 11:10:05 UTC 2014
#5817: Additional styling support for BuddyPress Widgets
--------------------------+-----------------------------
Reporter: mercime | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Theme | Version:
Severity: normal | Keywords:
--------------------------+-----------------------------
Currently, {{{buddypress.css}}} styles BP widgets on the premise that
theme authors are registering their sidebars like the WP default themes
do, where variables can be replaced with the necessary hooks like so:
{{{
register_sidebar( array(
...
'before_widget' => '<aside id="%1$s" class="widget %2$s">',
...
) );
}}}
However, there are theme authors which do not follow that procedure when
registering sidebars. Therefore, many BP widgets are not styled properly
because the ID's and classes required are not inserted in the markup. For
example, the Grisaille theme from the WP theme repo registers sidebars as
follows:
{{{
register_sidebar(
array(
...
'before_widget' => '<div class="sidebaritem">',
...
)
) );
}}}
Therefore the selector founds in buddypress.css, for example
{{{.widget.buddypress div.item-avatar img.avatar}}}, are not styling BP
widgets since the {{{widget}}} and {{{buddypress}}} classes are not
generated in the Grisaille theme's widget container.
Attached image shows the unstyled widgets of the Grisaille theme. I
included the Twenty Eleven because the disc list style was showing up.
Patch will follow.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5817>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list