Cool! Feedback!
—
@johnjamesjacoby
* If a member has 0 followers or is following 0 people, the nav tabs don’t appear.
– The original version did not show those tabs if it was empty so I decided to keep it that way at least for v1.1. Will change this for v1.1.1.
* “Users following me” and “Users I’m following” makes me think too much
– Gotcha. I’m changing this to “Following (COUNT)” and “Followers (COUNT)” for the WP Admin Bar in the next version to coincide with the point above.
* The toolbar top level menu feels out of place
* If you want to keep the “Follow” top level menu, maybe try to position it under “Friends” so it stays in the same order as the tabs
– This isn’t currently possible in BP. See ticket #3769.
* Maybe add “Followers” and “Following” as children of activity
– Currently, “Following” is already a subnav tab of activity only when the user is following though. I’ve addressed this so it will show all the time in the development version. “Followers” isn’t shown at all. If enough people request this one, I’ll add it.
* The activity stream query seems a bit slow. I haven’t looked at this code in months, but maybe there are some optimizations that can be made to the query.
– It’s probably a little slow on large installations like testbp.org. Should probably test along side the friend component’s activity since they both should be using similar code, but I agree that the activity stream query needs to be looked at.
* Notifications don’t seem to want to clear.
Can’t really duplicate this one locally, but I do seem to have the problem on testbp.org. Will investigate. If anyone can list the steps to duplicate this bug, that would be great!
—
@frizzo – Thanks for reporting this bug. Follow buttons did not work when javascript was disabled. This is fixed in the development version:
https://github.com/r-a-y/buddypress-followers/zipball/master
—
@qrahaman – You can modify the templates yourself by copying /buddypress-followers/_inc/templates/ directly to your WP theme. About consolidating tabs, I’m not sure if I will add this to the plugin. Some people will want it at the top level, some will want them consolidated like yourself. You could unhook the current “bp_follow_setup_nav” code and then run your own code to consolidate the tabs. The code is in the plugin.
—
@xevo – I have an idea about how to approach privacy for BP Follow, but it’s rather cumbersome. This will probably not get added for some time.
—
@unknown89 – You’re right that the buttons overlap in the directory. I was thinking of adding some inline CSS to the plugin, but thought it’s more of a general bp-default theme issue. I felt that it should be easy enough for theme devs to add a few lines of CSS themselves.
—
@imjscn – Plugins need to add support for BP Follow. As an example, etivite recently updated one of his plugins, Restrict Messages, to add BP Follow support. You can push other plugin devs to add support if they choose to, but it’s not my responsibility to sway them 