[buddypress-trac] [BuddyPress Trac] #6695: xProfile Draggable Admin UI needs the fieldsets to have ids

buddypress-trac noreply at wordpress.org
Fri Oct 30 18:21:54 UTC 2015


#6695: xProfile Draggable Admin UI needs the fieldsets to have ids
----------------------------------+-----------------------
 Reporter:  imath                 |      Owner:
     Type:  defect (bug)          |     Status:  new
 Priority:  normal                |  Milestone:  2.4
Component:  Component - XProfile  |    Version:
 Severity:  normal                |   Keywords:  has-patch
----------------------------------+-----------------------
 As @dimensionmedia reported on slack, it appears r10071 introduced a
 regression into the xProfile Draggable Admin UI :
 -> Ajax requests to set the order of fields or moving a field from a group
 of field to another failed because the `fieldset#id`s were missing.

 I suggest to keep the improvement of avoiding having duplicate ids on this
 page by prefixing the value of ids with a `draggable_` prefix and by
 editing the Ajax action so that it keeps on working the right way.

 The attached patch is my suggestion.

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


More information about the buddypress-trac mailing list