<html><body><div style="color:#000; background-color:#fff; font-family:tahoma, new york, times, serif;font-size:12pt">I see your point on it and not sure if it should be mandatory either.&nbsp; But since we do support the free themes on the support forums it would make it easier to do so.&nbsp; Was just a thought. :)<br><br>Any other feedback?<br><br>Aaron<br><div><span><br></span></div><div><br></div>  <div style="font-family: tahoma, new york, times, serif; font-size: 12pt;"> <div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <div dir="ltr"> <font face="Arial" size="2"> <hr size="1">  <b><span style="font-weight:bold;">From:</span></b> Edward Caissie &lt;edward.caissie@gmail.com&gt;<br> <b><span style="font-weight: bold;">To:</span></b> WP Theme Reviewers Mailing List &lt;theme-reviewers@lists.wordpress.org&gt; <br> <b><span style="font-weight: bold;">Sent:</span></b> Saturday, February 2, 2013 1:12 PM<br> <b><span
 style="font-weight: bold;">Subject:</span></b> Re: [theme-reviewers] Rule Addition?<br> </font> </div> <br>
<div id="yiv1190204102"><div dir="ltr"><div><div>I could see this as a recommendation and to a certain extent I am making it a personal coding standard to at least add code block termination comments throughout my most newest theme project (and will likely refactor my existing projects in a similar fashion).<br>

<br></div>I do not see this currently as being a required "standard" although I would be happy to re-visit the idea more thoroughly once the "Make" WordPress Inline Documentation guidelines are written and published for general use. I would also suggest putting this idea forward for potential inclusion with that "handbook" group.<br>

<br></div>As it is, I support the idea but not as a requirement and only as a reviewer's personal recommendation (and there is a difference between that and a RECOMMENDED review item).<br></div><div class="yiv1190204102gmail_extra">

<br clear="all"><div>Edward Caissie<br>aka Cais.</div>
<br><br><div class="yiv1190204102gmail_quote">On Sat, Feb 2, 2013 at 9:12 AM, Aaron Nimocks <span dir="ltr">&lt;<a rel="nofollow" ymailto="mailto:aaron_nimocks@yahoo.com" target="_blank" href="mailto:aaron_nimocks@yahoo.com">aaron_nimocks@yahoo.com</a>&gt;</span> wrote:<br><blockquote class="yiv1190204102gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div><div style="font-size:12pt;font-family:tahoma, new york, times, serif;"><div>I haven't gave this complete thought yet but I would assume this has been brought up before so just wanted to know if it has or why we don't do this.</div>

<div><br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">From a support or even end user perspective I would absolutely love if each template file was required to have a comment for when it started and ended.&nbsp; This way just viewing the source you can instantly tell which file it is in.&nbsp; I really don't think it would require too much effort for theme submitters but I think it would make a great deal of difference for supporting these themes as well as allowing end users to have a method of finding which page to edit.</div>

<div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;"><br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

Using something standard like</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;"><br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

&lt;!-- TEMPLATE FILE footer.php Start --&gt;</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">&lt;!-- TEMPLATE FILE footer.php End --&gt;</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

<br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">or for hooks</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

<br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">&lt;!-- TEMPLATE HOOK wp_footer Start --&gt;</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

&lt;!-- TEMPLATE HOOK wp_footer End --&gt;</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;"><br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

I just went through real quick and adding some on http://www.wpbum.com/how-to-get-jobs-on-odesk.html for just page template and only took a few minutes.&nbsp; But by viewing the source code it makes it really easy to see where all the code is being generated from and where to go to edit.<br>

</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;"><br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

Why isn't something like this a standard?</div><span class="yiv1190204102HOEnZb"><font color="#888888"><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;"><br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:tahoma, new york, times, serif;">

Aaron<br></div></font></span></div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a rel="nofollow" ymailto="mailto:theme-reviewers@lists.wordpress.org" target="_blank" href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
http://lists.wordpress.org/mailman/listinfo/theme-reviewers<br>
<br></blockquote></div><br></div>
</div><br>_______________________________________________<br>theme-reviewers mailing list<br><a ymailto="mailto:theme-reviewers@lists.wordpress.org" 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><br> </div> </div>  </div></body></html>