[theme-reviewers] Looking for feedback on theme review

[Chip Bennett]

Each of us has a slightly different approach. In my case, rather than merely
outputting everything from Theme-Check, I separate the Theme-Check output
into "Required" and "Recommended". I try to keep all the "Required" issues
together, and then put any recommendations after. And then I close out the
review with some boilerplate text/links.

As long as we're communicating effectively with the Theme developers
regarding what needs to be fixed in order to approve the Theme, that's what
ultimately matters.

Your review comments are all good comments. I like the way that you offered
suggested fixes, especially since, as you noted, all of the outstanding
issues are minor/easy to fix.

For a review such as that one, where the outstanding issues are all minor
and easy to fix, I have been experimenting with a new approach: I keep the
ticket open, and state that if an updated Theme with fixes is submitted
within a day or two, I'll close the current ticket as "newer version
uploaded" and proceed to review/approve the update with fixes.

(This approach seems to be doing well in my thus-far limited trial.)

Chip

On Wed, Mar 23, 2011 at 11:39 PM, Emil Uzelac <emil at themeid.com> wrote:

> You're welcome!
>
> *Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:
> emil at themeid.com | http://themeid.com
> Make everything as simple as possible, but not simpler. - Albert Einstein
>
>
>
> On Wed, Mar 23, 2011 at 11:25 PM, Ian Stewart <ian at iandanielstewart.com>wrote:
>
>> On 2011-03-23, at 11:15 PM, Emil Uzelac wrote:
>>
>> > Looks good Ian, well the only one thing that I would add to this would
>> be some references to WordPress guides such as:
>> >       • http://codex.wordpress.org/Theme_Review
>>
>> Thanks! I just added in a link to the Theme Unit Test right after my
>> mention of it.
>>
>> Ian
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>
>
> _______________________________________________
> 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/20110324/e916c2ea/attachment.htm>