[buddypress-trac] [BuddyPress Trac] #9308: Risky: bp-groups-template.php adds ".hidden" class.

[buddypress-trac]

#9308: Risky: bp-groups-template.php adds ".hidden" class.
--------------------------+------------------------------
 Reporter:  t.schwarz     |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Groups        |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by noruzzaman):

 I agree with this @dcavins  concern. Using a generic class name like
 .hidden is risky because it’s very likely to conflict with theme or
 framework CSS (Bootstrap, etc.).

 I’d be in favour of introducing prefixed classes like **group-status-
 public, group-status-private,** and **group-status-hidden,** even if that
 means a minor adjustment period for existing custom CSS. A short note in
 the release notes would be enough for developers to update their styles.

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