Phillip, welcome to the wonderful world of WordPress Theme Reviews ...<br><br>It's these subjective items that are more "difficult" to deal with than most technical issues that are found. Although they do get easier to sort through after you have finished several reviews.<br>
<br><br>Cais.<br><br><div class="gmail_quote">On Fri, Apr 15, 2011 at 11:02 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
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><font color="#888888">Chip</font><div><div></div><div class="h5"><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" target="_blank">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>On Fri, Apr 15, 2011 at 9:37 AM, Philip Walton <<a href="mailto:philip@philipwalton.com" target="_blank">philip@philipwalton.com</a>> wrote:<br>
</div><div>> 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>_______________________________________________<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>
</div></div></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>