[theme-reviewers] [theme-reviewer]

[Chip Bennett]

Please review the following instructions, found on the ticket request queue
page<http://make.wordpress.org/themes/about/trac-ticket-request-queue/trac-ticket-review-queue-march-2012/>
:

Based on recent ticket reviews, here are some things to keep in mind as you
perform your reviews:

   1. When reviewing a follow-up ticket, be sure that you check the
   previous ticket, and verify that all items indicated as “*required*”
   have been addressed. Summarize the fixed/not-fixed status of such items.
   2. Be sure that you clearly indicate what “*required*” issues are
   outstanding in your review, so that the developer can identify and address
   those issues more easily. This will also facilitate subsequent reviews.
   3. When running Themes through the Theme Check Plugin, you do not need
   to add review comments for “*recommended*” and “*info*” items. These are
   for your reference only, to facilitate your review. Note: you are more than
   welcome (and encouraged) to offer suggestions for improvements, as long as
   they are clearly identified as recommendations only, and not required.
   4. For Theme Unit Test-related comments, be sure to indicate the Post,
   Page, or context to which the comment/issue applies.

Regarding including files from within header.php, the header.php file is a
template-part file. I*f it is including other template-part files, it is
*required* to use get_template_part()*. Everything else that you've listed
from Theme Check is recommended only.

Thanks for your Theme-review contributions!

Chip

On Wed, Apr 4, 2012 at 2:39 AM, Elan Technosys <elantechnosys at gmail.com>wrote:

> I reviewed a theme and theme check is showing following message so do I
> accept or not.
>
>
>    - RECOMMENDED: No reference to post-thumbnails was found in the theme.
>    If the theme has a thumbnail like functionality, it should be implemented
>    with *add_theme_support( 'post-thumbnails' )*in the functions.php file.
>    - RECOMMENDED: No reference to *add_custom_image_header* was found in
>    the theme. It is recommended that the theme implement this functionality if
>    using an image for the header.
>    - RECOMMENDED: No reference to *add_custom_background()* was found in
>    the theme. If the theme uses background images or solid colors for the
>    background, then it is recommended that the theme implement this
>    functionality.
>    - *header.php* The theme appears to use include or require. If these
>    are being used to include separate sections of a template from independent
>    files, then *get_template_part()* should be used instead.
>
>    Line 134: <?php include ('slide-image.php')  ?>
>
>
>
>
> _______________________________________________
> 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/20120404/3dfe824e/attachment.htm>