<div dir="ltr">Yeah, code commenting is not a part of our guidelines. No need to worry over it too much.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 31, 2014 at 7:54 AM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I don't agree with the statement as a blanket idea ... it should deal with the intent going forward and should be removed and/or modified to reflect that idea.<div>
<br></div><div>IF the code is simply not working but is intended to be included in a future release, leaving it in as a noted "work in progress" item would be fine as far as I am concerned. This is also given that once the code is sorted out it will not be creating an instance where the theme should not be approved because of its implementation. The codex statement does not allow for this "style" of programming which we are not here to enforce, in most cases we can only recommend that best practices are followed.</div>
<span class="HOEnZb"><font color="#888888">
</font></span></div><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all"><div>Edward Caissie<br>aka Cais.</div></font></span><div><div class="h5">
<br><br><div class="gmail_quote">On Thu, Jul 31, 2014 at 7:10 AM, Stephen Cui <span dir="ltr"><<a href="mailto:scui2005@gmail.com" target="_blank">scui2005@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="auto"><div>These are documentation not guidelines. I do not think the statement is accurate.<span><font color="#888888"><br><br>Stephen </font></span></div><div><div><div><br>On Jul 31, 2014, at 6:52 AM, Thomas Ireland <<a href="mailto:info@designchapter.com" target="_blank">info@designchapter.com</a>> wrote:<br>
<br></div><blockquote type="cite"><div>
Not allowed... Theme codex -
<a href="http://codex.wordpress.org/Commenting_Code" target="_blank">http://codex.wordpress.org/Commenting_Code</a><br>
<br>
"Note: If you are submitting Theme to WordPress Directory, code
commenting should not be used."<br>
<br>
<div>On 7/31/2014 11:01 AM, Michael van den
Berg wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Hi
everyone!</span>
<div style="font-family:arial,sans-serif;font-size:13px"><br>
</div>
<div style="font-family:arial,sans-serif;font-size:13px">I'm
reviewing a theme which threw a few errors. The theme author
solved this issue by commenting a few lines out. Is this
acceptable? I guess it works, but it looks a bit messy...</div>
<div style="font-family:arial,sans-serif;font-size:13px"><br>
</div>
<div style="font-family:arial,sans-serif;font-size:13px">Thanks!</div>
<div style="font-family:arial,sans-serif;font-size:13px"><br>
</div>
<div style="font-family:arial,sans-serif;font-size:13px">
Best regards,</div>
<div style="font-family:arial,sans-serif;font-size:13px"><br>
</div>
<div style="font-family:arial,sans-serif;font-size:13px">Michael</div>
</div>
<br>
<fieldset></fieldset>
<br>
<pre>_______________________________________________
theme-reviewers mailing list
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
<br>
</div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>theme-reviewers mailing list</span><br><span><a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a></span><br>
<span><a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a></span><br></div></blockquote></div></div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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></div></div>
<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>