[theme-reviewers] Theme Uploader and CSS Classes

Chip Bennett chip at chipbennett.net
Fri Sep 24 19:40:10 UTC 2010

Also, I really think the uploader should output ALL failing criteria, rather
than stopping at the first one.

How frustrating it must be, to have to fix a Theme five or six times,
re-package it, and re-attempt to upload it. No wonder the Theme developers
get to us already frustrated!

Otto, can this be fixed? Can you set the upload script to complete ALL of
its checks, and then output the results, rather than stopping at the first
failed criterion?


On Fri, Sep 24, 2010 at 2:37 PM, Chip Bennett <chip at chipbennett.net> wrote:

> I think this is a bit of a fail: the uploader fails if it does not find the
> alignment classes in style.css.
> I understand the need to ensure that Themes include alignment classes, but
> this is part of the Review Guidelines. The uploader fails if it does not
> find these classes in style.css - but fails to check any other CSS file
> (that might, e.g. be @import included) to see if one of them includes the
> alignment classes.
> The easy fix, of course, is to add the required classes to style.css, and
> then override them later. But I think this is perhaps something that
> shouldn't fail the upload process.
> Thoughts?
> Chip
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20100924/cf0426b4/attachment.htm>

More information about the theme-reviewers mailing list