<html><body><div style="color:#000; background-color:#fff; font-family:times new roman, new york, times, serif;font-size:14pt"><div><span>Good point... I agree</span></div><div><br></div> <div style="font-family: 'times new roman', 'new york', times, serif; font-size: 14pt;"> <div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"> <div dir="ltr"> <hr size="1"> <font size="2" face="Arial"> <b><span style="font-weight:bold;">From:</span></b> Chip Bennett <chip@chipbennett.net><br> <b><span style="font-weight: bold;">To:</span></b> [theme-reviewers] <theme-reviewers@lists.wordpress.org> <br> <b><span style="font-weight: bold;">Sent:</span></b> Thursday, August 22, 2013 2:06 PM<br> <b><span style="font-weight: bold;">Subject:</span></b> [theme-reviewers] Enqueueing comment-reply<br> </font> </div> <div class="y_msg_container"><br>
<div id="yiv1003622335"><div dir="ltr">Good morning, reviewers,<div><br></div><div>I've noticed a few times now that reviewers have required Theme developers not to use 'comment_form_before' as the action into which to hook the callback to enqueue 'comment-reply'.</div>
<div><br></div><div>Hooking into 'comment_form_before' is perfectly fine - and in fact, using it actually eliminates one of the conditionals normally used, comments_open(), since the 'comment_form_before' action will never fire if comments are closed.</div>
<div><br></div><div>In general, WordPress will throw a _doing_it_wrong() if someone is attempting to call wp_enqueue_script() at an incorrect action. So, if you're not getting a _doing_it_wrong(), then please double-check before requiring the developer to change the action being used.</div>
<div><br></div><div>Thanks,</div><div><br></div><div>Chip</div></div>
</div><br>_______________________________________________<br>theme-reviewers mailing list<br><a ymailto="mailto:theme-reviewers@lists.wordpress.org" 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><br></div> </div> </div> </div></body></html>