[theme-reviewers] add_theme_support()

Chip Bennett chip at chipbennett.net
Mon Sep 20 00:48:17 UTC 2010


The Guidelines should be in good shape already. I made sure the correct
function calls were there already, and added the verbiage to the
add_theme_support() Codex page regarding Themes not making direct calls for
those arguments.

Chip

On Sun, Sep 19, 2010 at 7:22 PM, Edward Caissie <edward.caissie at gmail.com>wrote:

> Thanks for the "heads up", Nacin;
>
> We'll look into referencing this on the Theme Review page(s), too.
>
>
> Cais.
>
> On Sun, Sep 19, 2010 at 8:16 PM, Andrew Nacin <wp at andrewnacin.com> wrote:
>
>> A heads up, I've removed references to read-only parameters for
>> add_theme_support() from
>> http://codex.wordpress.org/Function_Reference/add_theme_support.
>>
>> Any theme enabling 'editor-style', 'menus', 'custom-header',
>> 'custom-background', or 'widgets' via add_theme_support() is doing it wrong.
>> These are internal registrations so we can transparently use
>> current_theme_support(). In fact WordPress prevents these in particular from
>> being removed via remove_theme_support(), that way there are no imbalances.
>>
>> Nacin
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20100919/4d536f9b/attachment.htm>


More information about the theme-reviewers mailing list