Nice adaptation to look for comment-reply only.<br><br><br clear="all">Cais.<br>
<br><br><div class="gmail_quote">On Wed, Jul 13, 2011 at 3:38 PM, Simon Prosser <span dir="ltr"><<a href="mailto:pross@pross.org.uk">pross@pross.org.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 13 July 2011 20:05, Otto <<a href="mailto:otto@ottodestruct.com">otto@ottodestruct.com</a>> wrote:<br>
> On Wed, Jul 13, 2011 at 1:57 PM, Chip Bennett <<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>> wrote:<br>
>> Perhaps make it an "INFO" if "comment-reply" is found, but not in the<br>
>> expected location? That way, the reviewer is flagged to verify the<br>
>> implementation?<br>
><br>
> Yep, had the same thought here. I just wrote a comment reply specific<br>
> check. It looks for the full method, then if it doesn't find that, it<br>
> looks for just "comment-reply" anywhere. If neither is found, that's a<br>
> fail. If only the "comment-reply" is found, then it generates an INFO<br>
> with:<br>
><br>
> "INFO: Could not find the comment-reply script enqueued, however a<br>
> reference to 'comment-reply' was found. Make sure that the<br>
> comment-reply script is being enqueued properly on singular pages."<br>
><br>
> -Otto<br>
<br>
</div>Thanks Otto, btw love the plugin diff emails!<br>
<div class="im"><br>
<br>
<br>
--<br>
My Blog: <a href="http://www.pross.org.uk/" target="_blank">http://www.pross.org.uk/</a><br>
Plugins : <a href="http://www.pross.org.uk/plugins/" target="_blank">http://www.pross.org.uk/plugins/</a><br>
Themes: <a href="http://wordpress.org/extend/themes/profile/pross" target="_blank">http://wordpress.org/extend/themes/profile/pross</a><br>
_______________________________________________<br>
</div><div><div></div><div class="h5">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>
</div></div></blockquote></div><br>