I don&#39;t know that all of the Google Font API-available fonts are GPL-compatible...<div><br></div><div>Chip<br><br><div class="gmail_quote">On Sat, Dec 24, 2011 at 11:54 AM, Edward Caissie <span dir="ltr">&lt;<a href="mailto:edward.caissie@gmail.com">edward.caissie@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Why not include, from your example, the Google Font in the package? Is there a benefit to making an API call versus referencing a locally available resource?<br>
<br><br clear="all">Cais.<div class="HOEnZb"><div class="h5"><br>
<br><br><div class="gmail_quote">On Sat, Dec 24, 2011 at 11:22 AM, Chip Bennett <span dir="ltr">&lt;<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


I&#39;m wondering why it would be a *bad* thing to say that all Themes must be self-contained, and should not hotlink any resources?<div><br></div><div>(I think API references are okay; e.g. Google Fonts - if they&#39;re not there, the CSS will fall back to another font, so the experience might be degraded, but it will degrade mostly gracefully.)</div>


<span><font color="#888888">
<div><br></div></font></span><div><span><font color="#888888">Chip</font></span><div><div><br><br><div class="gmail_quote">On Sat, Dec 24, 2011 at 9:56 AM, Edward Caissie <span dir="ltr">&lt;<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@Kirk - Leave a comment to the effect of what Otto has suggested why this method should not be used.<br><br>@Otto et al. - This should go into the Guidelines as *not* &quot;best practice&quot;, again for the same reason. &quot;Banning&quot; seems like such a harsh word to use, I&#39;m thinking simply not-approve-able due to the potential for &quot;common&quot; end-user conditions to cause the theme to simply not work, i.e.: intranet installation.<br>





<br>I&#39;m thinking something along these lines for the Guidelines: Themes must be self-contained within themselves and/or the WordPress core functionality. For example, externally referenced files may not always be available to the end-user and therefore should not be used.<br>





<br>The question is now where to put that into the guidelines as I recommend it become effective with the &quot;new&quot; 3.3 guidelines. Any one have any suggestions?<br><br><br clear="all">Cais.<div><div>
<br>
<br><br><div class="gmail_quote">On Fri, Dec 23, 2011 at 7:42 PM, Otto <span dir="ltr">&lt;<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div>On Fri, Dec 23, 2011 at 4:32 PM, Kirk Wight &lt;<a href="mailto:kwight@kwight.ca" target="_blank">kwight@kwight.ca</a>&gt; wrote:<br>
&gt; In particular, the theme uses a bunch of external APIs for the post formats;<br>
&gt; I&#39;ve searched the mailing list archives, and didn&#39;t find anything explicitly<br>
&gt; banning it, but, well, it weirds me out. A look at how the options are<br>
&gt; implemented would be appreciated too.<br>
<br>
</div>While there&#39;s nothing wrong with the files he&#39;s including and the<br>
sources are trustworthy enough, these files should be included in the<br>
theme and not added directly from external sites.<br>
<br>
In particular:<br>
<br>
wp_register_style(&#39;html5reset&#39;,<br>
&#39;<a href="http://html5resetcss.googlecode.com/files/html5-reset-1.4.css" target="_blank">http://html5resetcss.googlecode.com/files/html5-reset-1.4.css</a>&#39;,false,$theme_data[&#39;Version&#39;]);<br>
wp_enqueue_style( &#39;html5reset&#39;);<br>
wp_register_script(&#39;yui-css&#39;,&#39;<a href="http://yui.yahooapis.com/2.8.0r4/build/yuiloader/yuiloader-min.js" target="_blank">http://yui.yahooapis.com/2.8.0r4/build/yuiloader/yuiloader-min.js</a>&#39;,false,&#39;2.8.0r4&#39;);<br>






wp_enqueue_script(&#39;yui-css&#39;);<br>
wp_register_script(&#39;jquery-template&#39;,<br>
&#39;<a href="http://nje.github.com/jquery-tmpl/jquery.tmpl.js" target="_blank">http://nje.github.com/jquery-tmpl/jquery.tmpl.js</a>&#39;, array(&#39;jquery&#39;),<br>
&#39;0.1&#39;);<br>
wp_enqueue_script(&#39;jquery-template&#39;);<br>
<br>
This is bad because it adds a dependency on those sites. This means<br>
the theme won&#39;t work for some cases, such as on an internal intranet<br>
where the user viewing the site has no access to the public internet<br>
(this is a *far* more common use case than you might think).<br>
<br>
While there&#39;s no explicit guidelines prohibiting it that I found in<br>
the theme review list, I kinda sorta think that it should be banned.<br>
There&#39;s no good reason a theme can&#39;t simply package up these libraries<br>
in the theme, presuming the licenses on the libraries is compatible.<br>
YUI is BSD licensed, JQuery Template is MIT or GPLv2, and the reset<br>
CSS is public domain. All compatible.<br>
<span><font color="#888888"><br>
-Otto<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>
</font></span></blockquote></div><br>
</div></div><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>
<br></blockquote></div><br></div></div></div>
<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>
<br></blockquote></div><br>
</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></div>