[buddypress-trac] [BuddyPress Trac] #3278: @mentions autosuggest support

buddypress-trac noreply at wordpress.org
Thu Jul 3 18:53:20 UTC 2014


#3278: @mentions autosuggest support
---------------------------+-----------------------
 Reporter:  DJPaul         |       Owner:
     Type:  enhancement    |      Status:  assigned
 Priority:  normal         |   Milestone:  2.1
Component:  Notifications  |     Version:
 Severity:  normal         |  Resolution:
 Keywords:  dev-feedback   |
---------------------------+-----------------------

Comment (by DJPaul):

 Boone, I'm not totally sold by the suggestion of using a string parameter
 to provide a class name. It feels a bit weird as far as design patterns go
 in the WordPress world, and it could introduce unnecessary backwards
 compatibility restraints if we ever decided to rename things in the
 future.

 The `bp_suggestions_services` filter itself doesn't/isn't intended to do
 anything like providing a list of registered services; it's just there as
 a place for other plugins to add filters to, to add a new suggestions
 service. If anything, the older/first patch's approach to registering
 services was much more structured than this. I think I'm going to commit
 as-is for now, but as always, your feedback is greatly appreciated.

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


More information about the buddypress-trac mailing list