[buddypress-trac] [BuddyPress Trac] #5740: Description sould be part of the $field_type->edit_field_html() ? / scope of this function
buddypress-trac
noreply at wordpress.org
Wed Jul 9 01:39:46 UTC 2014
#5740: Description sould be part of the $field_type->edit_field_html() ? / scope
of this function
-------------------------+-----------------------------
Reporter: WCUADD | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Future Release
Component: XProfile | Version:
Severity: normal | Resolution:
Keywords: |
-------------------------+-----------------------------
Changes (by boonebgorges):
* milestone: Awaiting Review => Future Release
Comment:
I like this idea - I agree that it makes more sense for this markup to be
within the scope of the `BP_XProfile_Field_Type` class, if not the
`edit_field_html()` method itself. But this is going to be a very
difficult thing to implement, because the description and visibility
markup is currently called directly from the templates. So we'd have to
remove it from the templates. But then anyone who is overriding the core
templates will see the fields twice.
For this reason, I think the suggestion, while a good one, will have to
wait until we do a rewrite of the xprofile editing templates, and do a
full break of backward compatibility with bp-legacy. See #4952 for ongoing
discussion.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5740#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list