Just a thought ...<br><br>IF 'template_redirect' is being co-opted into this role, could not a new more appropriately named hook be considered, perhaps something along the lines of: 'pre_frontend' ... which may lead to a potential 'post_frontend' as well? Even something prefixed with 'before_' and 'after_' would be great.<br>
<br>I'm just thinking something that has a slightly more intuitive name convention rather than relying on an in-depth recognition of hook/action priority sequences.<br><br><br>Cais.<br><br><div class="gmail_quote">On Wed, Sep 15, 2010 at 11:35 AM, Andrew Nacin <span dir="ltr"><<a href="mailto:wp@andrewnacin.com">wp@andrewnacin.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><br><br><div class="gmail_quote"><div class="im">On Wed, Sep 15, 2010 at 11:31 AM, Austin Matzko <span dir="ltr"><<a href="mailto:austin@pressedcode.com" target="_blank">austin@pressedcode.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div>On Wed, Sep 15, 2010 at 10:21 AM, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>> wrote:<br>
> But, what we really need is what the official best-practice suggestion will<br>
> be, for the purposes of the Guidelines. We need to ensure that all reviewers<br>
> will be making consistent suggestions with respect to wp_enqueue_script()<br>
> within functions.php.<br>
<br>
</div>It's really a range depending on what you're trying to do: before init<br>
is too early; default wp_head is too late.<br>
<br>
I would strongly recommend that front-end scripts not be enqueued in<br>
the admin, so that means checking that is_admin() returns false if a<br>
front-end conditional "tag" like is_singular() isn't already used.</blockquote><div><br></div></div><div>The point is, before init is too early, but on init means it'll show up in the admin.</div><div>Hence template_redirect is a good option, as that is enough to ensure it only</div>
<div>gets enqueued for the frontend, with no is_admin() check required.</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>