[theme-reviewers] Custom javascript includes

Emil Uzelac emil at themeid.com
Mon Feb 20 07:37:36 UTC 2012

You're correct, as it says:

/* Always have wp_footer() just before the closing </body>
* tag of your theme, or you will break many plugins, which
* generally use this hook to reference JavaScript files.*/

Hook means from i.e. functions.php, no inline JS, author needs to
use wp_enqueue_scripts instead:

While going over footer.php in Theme in question I've noticed this:


styles must be moved to style.css and never added to DIVs directly. Another
"bad" practice is <div class='clearfix'></div> too, however some will let
this pass because it's not trivial issue, I personally would not recommend

Also see: http://nicolasgallagher.com/micro-clearfix-hack/


On Mon, Feb 20, 2012 at 12:57 AM, Paul Appleyard <paul at spacecat.com> wrote:

> Hi all
> The author is including some custom javascript at the end of the footer -
> the script is located within the theme directory. Does this need to be
> enqueued also? I'm also thinking he might need to enclose or namespace the
> functions here a little as well .. one of the (javascript) functions,
> light_box(), seems in danger of clashing with other plugins and such at
> some stage.
> Paul
> ______________________________**_________________
> theme-reviewers mailing list
> theme-reviewers at lists.**wordpress.org<theme-reviewers at lists.wordpress.org>
> http://lists.wordpress.org/**mailman/listinfo/theme-**reviewers<http://lists.wordpress.org/mailman/listinfo/theme-reviewers>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20120220/5fca0e82/attachment.htm>

More information about the theme-reviewers mailing list