As for "design choice": we try to give as much leeway as feasibly possible to the designer. Our primary requirement is that such design choices must be documented, i.e. in "readme.txt", so that the end users knows how to use the Theme properly.<div>
<br></div><div>Chip<br><br><div class="gmail_quote">On Fri, Apr 15, 2011 at 9:55 AM, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com">otto@ottodestruct.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Fri, Apr 15, 2011 at 9:37 AM, Philip Walton <<a href="mailto:philip@philipwalton.com">philip@philipwalton.com</a>> wrote:<br>
</div><div class="im">> Also, if I have a lot of categories, only the first 4<br>
> show up in the primary navigation menu.<br>
<br>
</div>There's a difference between "broken site" and "design choice". If the<br>
theme is designed to intentionally only allow 4 entries there, then<br>
that's okay. If it displays something visibly broken when you exceed 4<br>
entries (like cut off text, etc), then that's not okay.<br>
<br>
Whether a "little" problem is cause for failing the theme or not is<br>
more of a subjective call. For example, if an absurdly long post title<br>
causes a weird result, then that should be noted, but if it's the<br>
*only* problem then it may just be something to tell the theme<br>
developer to fix in the next version. On the other hand, if there are<br>
several different "little" problems like this, then you may decide<br>
that yes, this is too many, so they need to be fixed in order to be<br>
approved. At what point those "little" problems accumulate into a<br>
"fail" is up to you.<br>
<font color="#888888"><br>
-Otto<br>
</font><div><div></div><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br></div>