<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>No worries. This can indeed be a little confusing. I was also very confused when I started.<br><br>Sent from my iPhone</div><div><br>On 04-Oct-2014, at 2:00 pm, Liane Blanco <<a href="mailto:lianeblanco@gmail.com">lianeblanco@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">Okay, we're not on the same page. Literally. This is where I've been looking:<br><br><a href="https://make.wordpress.org/docs/theme-developer-handbook/part-four-releasing-your-theme/theme-review-guidelines/#code-quality">https://make.wordpress.org/docs/theme-developer-handbook/part-four-releasing-your-theme/theme-review-guidelines/#code-quality</a><br><div><br></div><div>which says:<br><br><span style="color:rgb(85,85,85);font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:16px;line-height:13px">"Themes must not generate any WordPress deprecated-function notices, PHP errors, warnings, or notices, <b>HTML/CSS validation errors</b>, or JavaScript errors."<br></span><br>I was looking in the theme developer handbook, not the theme reviewer handbook. Rohit, my apologies for contradicting you! :(</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 4, 2014 at 1:19 AM, Ulrich Pogson <span dir="ltr"><<a href="mailto:grapplerulrich@gmail.com" target="_blank">grapplerulrich@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">"Themes must not have any notices, warnings, or errors." This is under the section of PHP, CSS, JS and HTML.<br><div><br></div><div><a href="https://make.wordpress.org/themes/handbook/guidelines/code-quality/#php-js-css-html" target="_blank">https://make.wordpress.org/themes/handbook/guidelines/code-quality/#php-js-css-html</a><br></div><div><br></div><div>But validation is not needed as vendor specific css would not pass.</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 4 October 2014 10:16, Rohit Tripathi <span dir="ltr"><<a href="mailto:rohitink@live.com" target="_blank">rohitink@live.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>I am the mentor for Liane. And as far as I know, HTML/CSS validation is not required. But, it's good to get it confirmed once more.<br><br>Sent from my iPhone</div><div><div><div><br>On 04-Oct-2014, at 1:32 pm, Liane Blanco <<a href="mailto:lianeblanco@gmail.com" target="_blank">lianeblanco@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">While we're on the subject of errors, the Theme Guidelines do state that there should be no HTML/CSS errors. Does this still apply? I discussed with my mentor, who said that he was told that it was not important. So should that not be removed from the guidelines?<div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 4, 2014 at 12:42 AM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@uzelac.me" target="_blank">emil@uzelac.me</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Anytime you have the issue like that please Cc me directly in the ticket and I will resolve the problem :)<div><div><span></span><br><br>On Saturday, October 4, 2014, Liane Blanco <<a href="mailto:lianeblanco@gmail.com" target="_blank">lianeblanco@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yep, I figured it was something like that. :)</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 4, 2014 at 12:37 AM, Jose Castaneda <span dir="ltr"><<a>jomcastaneda@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>It was recently updated so if any themes were uploaded prior to that weren't caught. :)<br><br><br></div><div><div><div><br>On Oct 4, 2014, at 12:11 AM, Liane Blanco <<a>lianeblanco@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">It must not have, or something changed between the time the theme was uploaded, because I had two errors about screen_icon being deprecated since 3.8, and a lot of recommends. I pasted these for the theme author, and also linked to the validation results for his CSS stylesheet (there were 40 errors) and he got angry and said he wanted another reviewer. Kind of disconcerting...<br><br>Anyway, live and learn. I will go over each file more carefully in future. Should I just not tell the author in future that there are recommends from the Theme Check?</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 3, 2014 at 11:51 PM, <span dir="ltr"><<a>tskk79@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">You should do both, run themecheck and go line by line, but if there are any required issues by themecheck, theme upload will fail anyway.<br>
Sent from BlackBerry® on Airtel<br>
<div><div><br>
-----Original Message-----<br>
From: Liane Blanco <<a>lianeblanco@gmail.com</a>><br>
Sender: "theme-reviewers" <<a>theme-reviewers-bounces@lists.wordpress.org</a>>Date: Fri, 3 Oct 2014 23:48:03<br>
To: Discussion list for WordPress theme reviewers.<<a>theme-reviewers@lists.wordpress.org</a>><br>
Reply-To: "Discussion list for WordPress theme reviewers."<br>
<<a>theme-reviewers@lists.wordpress.org</a>><br>
Subject: Re: [theme-reviewers] Theme Review Question - finding errors<br>
<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a>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>
theme-reviewers mailing list<br>
<a>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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Insert clever phrase here
</div>
</div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>theme-reviewers mailing list</span><br><span><a>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>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><br clear="all"><div><br></div>-- <br>Insert clever phrase here
</div>
</blockquote>
</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><br clear="all"><div><br></div>-- <br>Insert clever phrase here
</div>
</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><br clear="all"><div><br></div>-- <br>Insert clever phrase here
</div>
</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">theme-reviewers@lists.wordpress.org</a></span><br><span><a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a></span><br></div></blockquote></body></html>