[buddypress-trac] [BuddyPress] #2742: Allow group admins to choose a groups front page.
buddypress-trac at lists.automattic.com
buddypress-trac at lists.automattic.com
Sun Feb 27 16:33:50 UTC 2011
#2742: Allow group admins to choose a groups front page.
--------------------------+------------------------------------
Reporter: modemlooper | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 1.3
Component: Activity | Version: 1.3
Resolution: | Keywords: has-patch dev-feedback
--------------------------+------------------------------------
Changes (by boonebgorges):
* keywords: needs-patch => has-patch dev-feedback
Comment:
I took a swing at making home content configurable by user without having
to change theme files. See 2742.patch.
A couple considerations for when you look at this (as of yet only half-
baked) patch:
1) It adds a dropdown to group Admin > Group Settings. Seemed like the
logical place
2) To allow for maximum flexibility, especially in terms of group
extensions, I build the dropdown menu dynamically. But I realized when I
was done that this won't be good enough, because template names (eg
groups/single/activity.php) are not enough to differentiate between
multiple group extensions, which all use plugins.php and then populate the
page with hooks. So my method won't work exactly. It suggests two possible
solutions: (a) make it so that only core tabs (Members, Forum, Activity,
Send Invites) can be Home (easy but lame), or (b) intercept the Home
loading process higher up the chain so that things like current_action can
be swapped out. One possibility is for example.com/groups/group-name/ to
never actually resolve; it always gets redirected to the sub-page
(...group-name/activity, ...group-name/members) that is set as the group
home. Not sure what the SEO implications are for this move.
3) As it stands, the patch doesn't directly address modemlooper's original
goal, which was to allow arbitrary content on the Home page. But, when
combined with a very simple implementation of the BP Group Extension API -
one that has a static display() method and nothing else - it *does* solve
the problem, in a theme-independent way.
4) I haven't written the logic to deal with fallbacks when the selected
Home component is disabled. Should be fairly trivial. Allowing group
admins to disable the Activity tab should be trivial as well - just
another checkbox on the Group Settings page.
5) There is also the issue of which tabs are shown. Right now, the
Activity tab is not shown on the group, because it's assumed that Home =
Activity. As this patch stands, changing to Members (for example) makes
group activity inaccessible. I kinda think that we should show all active
tabs all the time, even if it means that two of them (eg Members and Home)
show the same content. Or maybe we just get rid of the Home tab
altogether.
Feedback welcome. Is this patch a step in the right direction? Or is a
more theme-specific approach more desirable?
--
Ticket URL: <https://trac.buddypress.org/ticket/2742#comment:2>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list