[buddypress-trac] [BuddyPress] #2598: [patch] Forum topic title should be included in bp_get_page_title()

[buddypress-trac at lists.automattic.com]

#2598: [patch] Forum topic title should be included in bp_get_page_title()
--------------------+-------------------------------------------------------
 Reporter:  r-a-y   |       Owner:                          
     Type:  defect  |      Status:  new                     
 Priority:  normal  |   Milestone:  1.2.6                   
Component:  Forums  |    Keywords:  has-patch, needs-testing
--------------------+-------------------------------------------------------

Comment(by hnla):

 My Opinion? Patch to Boone's 2958.3 patch (no justification in this
 holding things up)

 This addresses the main point of no topic title which is the real issue
 and gives us:

 Site Name | Groups | Group Name | Forum | Topic title  - for title tag
 string

 For me I would say it's perhaps too verbose and prefer a variation on
 Boon's forum thread example and will possibly still extract the function
 and add to it:

 $title = str_replace( ' | Groups', '', $title );
 $title = str_replace( ' | Forum', '| Topic', $title );

 But this is I think a matter of choice? up to the individual to decide if
 they wish or prefer something a little different.

 Hopefully we will start building a library of snippets functions people
 can use when the new codex is in place.

 As for my previous comment not sure whether this bp_page_title() should be
 added to 1.3 as an enhancement ticket with a view to checking through
 thoroughly? The question of the hyphen in post category titles is one I
 think will have to be tackled  with a user function for time being,
 shouldn't be difficult once the correct cat object 'name' or 'nice name'
 is found, did have a quick go but hadn't really time to hunt through WP
 codex.

-- 
Ticket URL: <http://trac.buddypress.org/ticket/2598#comment:17>
BuddyPress <http://buddypress.org/>
BuddyPress