If it fails the Theme Unit Tests, then those failures should be noted. If they are sufficient (a subjective evaluation), then the Theme should be failed.<div><br></div><div>(p.s. if you see things that you think should be mentioned in the Theme Unit Tests, but aren't, then please mention them here on the mail list, so that I can update the test data accordingly.)</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Fri, Apr 15, 2011 at 9:37 AM, Philip Walton <span dir="ltr"><<a href="mailto:philip@philipwalton.com">philip@philipwalton.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I'm reviewing a theme called zeeReputation (<a href="http://themes.trac.wordpress.org/ticket/3268" target="_blank">http://themes.trac.wordpress.org/ticket/3268</a>). It has no PHP errors, no HTML/CSS validation errors, it passes all the Theme Checker tests, and it calls no deprecated functions.<br>
<br>
However, it doesn't really adjust well to the different theme variables. For example, if the site title is relatively long, it wraps to the next line and covers up some links. Also, if I have a lot of categories, only the first 4 show up in the primary navigation menu.<br>
<br>
My question is this: are issues like these ground for rejecting a submission, or should I focus primarily on preventing errors and the like?<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>
</blockquote></div><br></div>