<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>I agree to Ulrich. But, Unfortunately there is no such naming convention for common jquery scripts.<br><br><div><hr id="stopSpelling">Date: Thu, 27 Feb 2014 16:02:46 +0100<br>From: grapplerulrich@gmail.com<br>To: theme-reviewers@lists.wordpress.org<br>Subject: Re: [theme-reviewers] Bundling libraries and prefixing<br><br><p dir="ltr">I would say no because if you have a plugin that uses novo slider also then you will load the script twice which can cause issues and is stupid to load the script twice. </p>
<p dir="ltr">There should be a standard naming convention. So if the script is dependent on Jquery it should be jquery-nivo-slider. </p>
<div class="ecxgmail_quote">On 27 Feb 2014 15:35, "Rohit Tripathi" <<a href="mailto:rohitink@live.com">rohitink@live.com</a>> wrote:<br><blockquote class="ecxgmail_quote" style="border-left:1px #ccc solid;padding-left:1ex;">



<div><div dir="ltr">It should be <div><br></div><div>wp enqueue_script( 'themename-nivo-slider', ... );<br><br>As the rule states, all theme defined hooks/functions should be properly prefixed.<br><br><div>> Date: Thu, 27 Feb 2014 08:32:38 -0600<br>
> From: <a href="mailto:kovshenin@gmail.com" target="_blank">kovshenin@gmail.com</a><br>> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>> Subject: [theme-reviewers] Bundling libraries and prefixing<br>
> <br>> Quick question.<br>> <br>> If a theme bundled the Twitter Bootstrap library, or Nivo Slider, or<br>> jQuery Colorbox, should it prefix the wp_enqueue_script/style handles<br>> with the theme name, or should it use a generic name?<br>
> <br>> wp_enqueue_script( 'nivo-slider', ... );<br>> <br>> vs<br>> <br>> wp enqueue_script( 'themename-nivo-slider', ... );<br>> <br>> Thanks!<br>> <br>> -- <br>> Konstantin<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>
</div></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>_______________________________________________
theme-reviewers mailing list
theme-reviewers@lists.wordpress.org
http://lists.wordpress.org/mailman/listinfo/theme-reviewers</div>                                           </div></body>
</html>