Also, I really think the uploader should output ALL failing criteria, rather than stopping at the first one.<div><br></div><div>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!</div>
<div><br></div><div>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?</div><div><br></div><div>Chip</div>
<div><br><div class="gmail_quote">On Fri, Sep 24, 2010 at 2:37 PM, 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;">
I think this is a bit of a fail: the uploader fails if it does not find the alignment classes in style.css.<div><br></div><div>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.</div>
<div><br></div><div>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.</div><div><br></div>
<div>Thoughts?</div><div><br></div><font color="#888888"><div>Chip</div>
</font></blockquote></div><br></div>