<font color="#333333"><font><font face="arial,helvetica,sans-serif">Agreed Chip, with an exception that this was previously approved Theme ;) But we got that cleared now.<br></font></font></font><br><div class="gmail_quote">
On Thu, Mar 8, 2012 at 5:19 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">
The reviewer was right in this case. The Theme *does* implement custom header and custom background, but uses a *custom* implementation, rather than supporting the core implementation.<div><br></div><div>As Amy suggested, though, it would probably be best for review-specific comments to be left in-ticket, for posterity and for future reference for the Theme in question.</div>
<div><br></div><div>Thanks,</div><div><br></div><div>Chip<div><div class="h5"><br><br><div class="gmail_quote">On Thu, Mar 8, 2012 at 5:07 PM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font color="#333333"><font><font face="arial,helvetica,sans-serif">No, Theme reviewer is not and should not be disconnected from this or any other ticket. Previously approved tickets should have at least 2 day grace period, where reviewer will place the Theme on hold till all requirements are fixed. If and when the Theme is not-approved it will not go back in #1 priority.</font></font></font><div>
<font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Theme is recommended to incorporate the following WordPress core features, but is not required to do so. However, if incorporated, features must support the core WordPress implementation: I hate to say this but it was wrong to say that custom header, background are required.</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">For this reason only ticket should be re-opened and make sure that <a href="http://codex.wordpress.org/Theme_Review#Theme_Features" target="_blank">http://codex.wordpress.org/Theme_Review#Theme_Features</a> is followed by the reviewer.</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Cheers,</font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Emil<br>
</font></font></font><br><div class="gmail_quote"><div>On Thu, Mar 8, 2012 at 4:56 PM, Mats Birch <span dir="ltr"><<a href="mailto:birch@birchware.se" target="_blank">birch@birchware.se</a>></span> wrote:<br>
</div><div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I've got a question for you, who know how these things work.<br>
<br>
I just upgraded my theme, and got a "not approved" review. I am a bit curious, because none of the things remarked upon have been changed since the previous version were approved, with no mentions of those things.<br>
<br>
How did that happen?<br>
<br>
I of course fixed, within an hour, most of it, and commented on the ticked, asking if I had done enough.<br>
I got no answer.<br>
Is that because a reviewer is disconnected from a ticket once they closed it?<br>
<br>
Anyway, I uploaded it anyway (what else to do?) and, much to my surprise, I found it in the "Priorty 3 queue", last of the last, so to speak. Wouldn't it be smoother if the reviewer could check the new one while he/she sort of knows it?<br>
<br>
<br>
The theme's last ticket is: <a href="http://themes.trac.wordpress.org/ticket/6839" target="_blank">http://themes.trac.wordpress.<u></u>org/ticket/6839</a><br>
<br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</blockquote></div></div></div><br></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div></div></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br>