[wp-trac] Re: [WordPress Trac] #9103: Put search string into <title> of search results page

WordPress Trac wp-trac at lists.automattic.com
Fri Feb 13 01:17:24 GMT 2009


#9103: Put search string into <title> of search results page
--------------------------+-------------------------------------------------
 Reporter:  jidanni       |       Owner:  anonymous              
     Type:  defect (bug)  |      Status:  new                    
 Priority:  lowest        |   Milestone:  2.8                    
Component:  Template      |     Version:  2.7                    
 Severity:  trivial       |    Keywords:  has-patch needs-testing
--------------------------+-------------------------------------------------

Comment(by jidanni):

 I believe I'm trying to say: instead of your diff's
 {{{
 $title = sprintf(__('Search Results %s %s'), $t_sep,
 wp_specialchars($search));
 }}}
 You should do
 {{{
 $title = sprintf(__('%s %s Search Results'), wp_specialchars($search),
 $t_sep,);
 }}}
 As Google and Wikipedia do. Else you will have munched 75% of each tab
 with "Search Results". Better still would be
 {{{
 $title = sprintf(__('%s %s Search'), wp_specialchars($search), $t_sep,);
 }}}

 Mine are for a totally "leftist" solution.
 It seems your original should also be retained, to provide for an
 additional totally "rightest" choice.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/9103#comment:6>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list