[buddypress-trac] [BuddyPress] #2975: Clean up bp-default header.php

Tue Dec 28 03:53:06 UTC 2010

#2975: Clean up bp-default header.php
-----------------------+-----------------------------
 Reporter:  r-a-y      |      Owner:
     Type:  defect     |     Status:  new
 Priority:  normal     |  Milestone:  Awaiting Review
Component:  Theme      |    Version:  1.3
 Keywords:  has-patch  |
-----------------------+-----------------------------
 The attached patch cleans up a few things in header.php of the bp-default
 theme.

 Specifically:
 - moves the stylesheet higher up the priority in the <head> tag

 - removes the <meta> generator Wordpress tag as this is already
 automatically added by Wordpress in the "wp_head" action.

 - removes the hardcoded activity feed links and attaches them to the
 "bp_head" action. [omission of whitespace is intentional in
 bp_dtheme_sitewide_feed()]

 - removes some unnecessary blank lines

 - *bonus* removes secondary blog avatars as this wasn't back-patched to
 trunk.

 ---

 Right now, I've added the feed links in /bp-default/functions.php, but
 might be better placed in /bp-activity.php.  Thoughts?

 Like #2764, if committed, child theme authors will have to remove the
 hardcoded feed links in their header.php to prevent double feeds from
 appearing in <head>.

-- 
Ticket URL: <https://trac.buddypress.org/ticket/2975>
BuddyPress <http://buddypress.org/>
BuddyPress