[buddypress-trac] [BuddyPress Trac] #7670: Let user pass additional param to edit_field_html() to show/hide form labels

[buddypress-trac]

#7670: Let user pass additional param to edit_field_html() to show/hide form
labels
------------------------------+-----------------------------
 Reporter:  hnla              |      Owner:
     Type:  enhancement       |     Status:  new
 Priority:  low               |  Milestone:  Awaiting Review
Component:  Extended Profile  |    Version:
 Severity:  minor             |   Keywords:
------------------------------+-----------------------------
 This relates to: #7665

 Currently we hardcode the screen reader class to the label elements in the
 classes.

 We have no easy means of changing this from the template  perspective, I
 would perhaps prefer to re-instate label visual display for Nouveau.

 The attached patch is a proof of concept and not considered the right
 approach, necessarily. The current param `array $raw_properties = array()`
 ( intended for form control key/value pairs ) feels odd and suitable for
 refactoring, however the manner in which this is processed makes things
 awkward, additional args slipped in to array are fetched prior to
 bp_parse_args() then unset from array, feels a little dirty to mix in this
 way but same approach could be used for  a new key pair `'show_label' =>
 true,`

 This only addresses one class file for datebox at this time.

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