Please review <a href="http://make.wordpress.org/themes/about/trac-ticket-request-queue/trac-ticket-review-queue-march-2012/">the following instructions, found on the ticket request queue page</a>:<div><br></div><blockquote class="webkit-indent-blockquote" style="margin:0 0 0 40px;border:none;padding:0px">
<div><span class="Apple-style-span" style="font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18px;background-color:rgb(255,255,255)"><p style="margin-top:0.7em;margin-right:0px;margin-bottom:0.7em;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px;color:rgb(85,85,85);font-size:1em;line-height:1.4em">
Based on recent ticket reviews, here are some things to keep in mind as you perform your reviews:</p><ol style="margin-top:16px;margin-right:0px!important;margin-bottom:22px!important;margin-left:18px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0.75em;list-style-type:decimal;list-style-position:initial;font-size:1em;line-height:1.4em;border-width:initial;border-color:initial;border-right-width:0px;border-bottom-width:0px;border-left-width:0px;border-right-style:none;border-bottom-style:none;border-left-style:none;border-top-style:none;border-top-width:0px;overflow-x:visible;overflow-y:visible">
<li style="color:rgb(85,85,85);margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:2px;padding-right:0px;padding-bottom:0px;padding-left:0px;overflow-x:visible;overflow-y:visible;border-top-style:none;border-width:initial;border-color:initial;border-right-width:0px;border-bottom-width:0px;border-left-width:0px;border-right-style:none;border-bottom-style:none;border-left-style:none;display:list-item">
When reviewing a follow-up ticket, be sure that you check the previous ticket, and verify that all items indicated as “<strong style="font-weight:bold">required</strong>” have been addressed. Summarize the fixed/not-fixed status of such items.</li>
<li style="color:rgb(85,85,85);margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:2px;padding-right:0px;padding-bottom:0px;padding-left:0px;overflow-x:visible;overflow-y:visible;border-top-style:none;border-width:initial;border-color:initial;border-right-width:0px;border-bottom-width:0px;border-left-width:0px;border-right-style:none;border-bottom-style:none;border-left-style:none;display:list-item">
Be sure that you clearly indicate what “<strong style="font-weight:bold">required</strong>” issues are outstanding in your review, so that the developer can identify and address those issues more easily. This will also facilitate subsequent reviews.</li>
<li style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:2px;padding-right:0px;padding-bottom:0px;padding-left:0px;overflow-x:visible;overflow-y:visible;border-top-style:none;border-width:initial;border-color:initial;border-right-width:0px;border-bottom-width:0px;border-left-width:0px;border-right-style:none;border-bottom-style:none;border-left-style:none;display:list-item">
<font class="Apple-style-span" color="#ff0000">When running Themes through the Theme Check Plugin, you do not need to add review comments for “<strong style="font-weight:bold">recommended</strong>” and “<strong style="font-weight:bold">info</strong>” 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.</font></li>
<li style="color:rgb(85,85,85);margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:2px;padding-right:0px;padding-bottom:0px;padding-left:0px;overflow-x:visible;overflow-y:visible;border-top-style:none;border-width:initial;border-color:initial;border-right-width:0px;border-bottom-width:0px;border-left-width:0px;border-right-style:none;border-bottom-style:none;border-left-style:none;display:list-item">
For Theme Unit Test-related comments, be sure to indicate the Post, Page, or context to which the comment/issue applies.</li></ol></span></div></blockquote><div>Regarding including files from within header.php, the header.php file is a template-part file. I<b>f it is including other template-part files, it is *required* to use get_template_part()</b>. Everything else that you've listed from Theme Check is recommended only.</div>
<div><br></div><div>Thanks for your Theme-review contributions!</div><div><br></div><div>Chip</div><div><br><div class="gmail_quote">On Wed, Apr 4, 2012 at 2:39 AM, Elan Technosys <span dir="ltr"><<a href="mailto:elantechnosys@gmail.com">elantechnosys@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I reviewed a theme and theme check is showing following message so do I accept or not.<br><br><ul><li><span>RECOMMENDED</span>: No
reference to post-thumbnails was found in the theme. If the theme has a
thumbnail like functionality, it should be implemented with <strong>add_theme_support( 'post-thumbnails' )</strong>in the functions.php file.</li><li><span>RECOMMENDED</span>: No reference to <strong>add_custom_image_header</strong> was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.</li>
<li><span>RECOMMENDED</span>: No reference to <strong>add_custom_background()</strong>
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.</li><li><strong>header.php</strong> The theme appears to use include or require.
If these are being used to include separate sections of a template from
independent files, then <strong>get_template_part()</strong> should be used instead. <pre>Line 134: <?php <span>include (</span>'slide-image.php') ?> </pre></li></ul><br>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>