[theme-reviewers] Theme Check update

[Edward Caissie]

I'm going to agree with this as well ...

On Thu, Sep 11, 2014 at 7:22 PM, Otto <otto at ottodestruct.com> wrote:

> On Thu, Sep 11, 2014 at 5:19 PM, Justin Tadlock <justin at justintadlock.com>
> wrote:
>>
>> 1) "The <title> tags can only contain a call to wp_title(). Use the
>> wp_title filter to modify the output."
>>
>> This is caused by a custom function hooked to `wp_head` to output the
>> `<title>` tag.  This is should be easy enough to fix for theme check
>> though.  Here's the code that gets flagged:
>>
>> printf( "<title>%s</title>\n", wp_title( ':', false ) );
>>
>
> IMO, the title should look like this, in the header.php, in all themes:
>
> <title><?php wp_title(whatever); ?></title>
>

I'm not really sure why one would want to use some sort of substitution
code like the example, just seems to make it much more complicated than
what it needs to be.




Edward Caissie
aka Cais.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20140911/3681db66/attachment.html>