<div class="gmail_quote">On Sun, Sep 12, 2010 at 6:03 PM, 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;">
I may not express this properly, given that I've had a couple of beers<br>
today, but... ;)<br>
<br>
When you give a theme not-approval, you're not actually stopping the<br>
theme from being released, or stopping people from using it, or<br>
anything of the sort. You're just cutting off the users of that theme<br>
from the primary update mechanism that is built right into WP.<br>
<br>
I've looked at several of the rejected themes in the trac, and without<br>
exception, every single one of them is available for download on the<br>
author's site.<br>
<br>
See, when a theme gets updated in the repo, users get a notification.<br>
They can one-click update the theme and get new features, security<br>
fixes, etc... Think of plugin updates, it's the same basic principle<br>
here.<br>
<br>
Given that themes are just as capable as plugins, this sort of thing<br>
isn't protecting users, it's actively harming them. It hurts users to<br>
be using themes that are not from <a href="http://wp.org" target="_blank">wp.org</a>. Sure, a theme author could<br>
implement their own update mechanism (it's actually quite easy), but<br>
none really have so far.<br></blockquote><div><br></div><div>We cannot control what code is available outside of the repo. That a developer chooses to release lower-quality code outside of the repo, rather than higher-quality code through the repo, is a choice that only he can make. That users choose to download and install code from outside the repo, rather than through the repo, is a choice that only they can make.</div>
<div><br></div><div>All those decisions are entirely outside the scope of the Theme Review process and team. We should not lower our quality standard because of those decisions.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
We *really* want users to get theme updates and to get used to<br>
updating themes in the same way that they update plugins. If this<br>
means we need to sacrifice a few minor things here and there then<br>
quite frankly I'm for letting the "minor" things slide.<br></blockquote><div><br></div><div>Fair enough. Spell out what guidelines you consider to be minor, and how many such minor infractions should be allowable for a passing Theme. </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
I have full faith in the theme reviewers to be able to determine what<br>
is a minor problem that can be fixed in an update vs. what is a major<br>
problem that must be resolved immediately. C'mon people, you have both<br>
brains and judgment skills. You're all programmers. You know PHP. You<br>
should be able to tell what's minor or not. This "abide by the list"<br>
mentality is not helpful to anybody.<br></blockquote><div><br></div><div>The Theme review queue is holding steady at more than 60 Themes. We don't have time to make the process more subjective and less objective. The "abide by the list" mentality ensures fairness and facilitates the most efficient, expeditious review process.</div>
<div><br></div><div>The way I see it, criteria should be *required*, *recommended*, or *optional*. If a required criterion fails, then it must be fixed before the Theme passes. If we have *required* criteria that should be *recommended*, then let's address those. </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
But that's just my opinion. Take it as you will. ;)<br>
<div><div></div><div class="h5"><br>
-Otto<br></div></div></blockquote><div><br></div><div>Any sincere, respectful opinion is appreciated and given due consideration. :)</div><div><br></div><div>Chip</div></div>