[buddypress-trac] [BuddyPress Trac] #5498: Add 'state' classes to visibility settings JS

[buddypress-trac]

#5498: Add 'state' classes to visibility settings JS
-------------------------+-----------------------------
 Reporter:  hnla         |      Owner:
     Type:  enhancement  |     Status:  new
 Priority:  low          |  Milestone:  Awaiting Review
Component:  Theme        |    Version:
 Severity:  minor        |   Keywords:  has-patch
-------------------------+-----------------------------
 Patch adds 'state' classes; 'open' to the .field-visibility-settings
 element  & 'hide' to .field-visibility-settings-toggle adding /removing
 them according to the click events.

 This enhancement allows CSS to 'know' what state the elements are in and
 to then be able to render styles accordingly e.g transitions, but in
 general increases the devs options in managing the elements.

 Ideally all control of visual states should be handed back to CSS with JS
 simply being used to set classes & CSS handling display or any other
 effects, however wasn't sure what the implications would be on established
 sites that may have overloaded styles and would suddenly lose the js
 functionality.

 Probably to late to squeeze into 2.0 but useful if we could push this into
 2.1?

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5498>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac