[theme-reviewers] How much should I weigh looks, scalability, and functionality?

[Chip Bennett]

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.

(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.)

Chip

On Fri, Apr 15, 2011 at 9:37 AM, Philip Walton <philip at philipwalton.com>wrote:

> I'm reviewing a theme called zeeReputation (
> http://themes.trac.wordpress.org/ticket/3268). It has no PHP errors, no
> HTML/CSS validation errors, it passes all the Theme Checker tests, and it
> calls no deprecated functions.
>
> 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.
>
> My question is this: are issues like these ground for rejecting a
> submission, or should I focus primarily on preventing errors and the like?
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110415/9839c441/attachment.htm>