[buddypress-trac] [BuddyPress Trac] #5914: Registration form: Proposed enhancements for touch devices

buddypress-trac noreply at wordpress.org
Wed Jan 7 21:22:59 UTC 2015


#5914: Registration form: Proposed enhancements for touch devices
--------------------------------------+------------------
 Reporter:  standardspace             |       Owner:
     Type:  enhancement               |      Status:  new
 Priority:  normal                    |   Milestone:  2.2
Component:  Theme / Template Parts    |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  good-first-bug has-patch  |
--------------------------------------+------------------
Changes (by r-a-y):

 * keywords:  good-first-bug has-patch commit => good-first-bug has-patch


Comment:

 When looking to add in XProfile support, I came across
 `BP_XProfile_Type::get_edit_field_html_elements()`, which does something
 similar to what was in `01.patch`.

 In order to avoid duplication of code, `02.patch` flip-flops the function
 name back to `bp_form_field_attributes()` and adds a second argument for
 attributes you want to pass for output.

 For XProfile, the field name is passed rather than the type.  For example,
 if I created a XProfile textbox field called "College", this is what the
 call looks like:

 `bp_get_form_field_attributes( 'college', $existing_attributes );`

 This makes more sense because if you passed `'textbox'`, you couldn't
 target individual fields.

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


More information about the buddypress-trac mailing list