<div dir="ltr">@Josh - Yes, I recognized the font would have a fallback safety net of the browser default font (at worst), but since it is so easy to bundle a GoogleFont with a theme I see no reason to call it outside of the theme. It's a very specific example but true none the less.<br>
</div><div class="gmail_extra"><br clear="all"><div>Edward Caissie<br>aka Cais.</div>
<br><br><div class="gmail_quote">On Tue, Aug 6, 2013 at 10:49 PM, Josh Pollock <span dir="ltr"><<a href="mailto:jpollock412@gmail.com" target="_blank">jpollock412@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">@edward I think the important distinction between calling a font from Google Font and a library from a CDN is that if the browser can't get the font, it will use a default web font instead, while an unavailable script will all most always result in failure to function.</div>
<div class="HOEnZb"><div class="h5">
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 6, 2013 at 3:37 PM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Personally I think "everything the theme calls that doesn't come from WordPress core *must* be bundled with the theme" without exception, including items such as GoogleFonts for the same reasons that @Otto mentioned, the installation may be found where there is potentially no direct access to the Internet by the "reader". Although if that is the case then the example GoogleFonts call would at worst fall back to the browser default(?) which one would hope still keeps the theme "readable" with just a loss of aesthetics.<span><font color="#888888"><br>
</font></span></div><div class="gmail_extra"><span><font color="#888888"><br clear="all"><div>Edward Caissie<br>aka Cais.</div></font></span><div><div>
<br><br><div class="gmail_quote">On Tue, Aug 6, 2013 at 3:15 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Everything the Theme calls that doesn't come from WordPress core *must* be bundled with the Theme.<div><br></div><div>Just about the only exception to that rule is an API call such as GoogleFonts.</div>
</div>
<div class="gmail_extra"><br><br><div class="gmail_quote"><div>On Tue, Aug 6, 2013 at 2:37 PM, Ola Łączek <span dir="ltr"><<a href="mailto:ola@bodera.com" target="_blank">ola@bodera.com</a>></span> wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
Hello!<div><br></div><div>Is it ok for theme author to enqueue scripts directly from CDN, and not include them with the theme?</div><div>Example: <script src="<a href="http://html5shim.googlecode.com/svn/trunk/html5.js" target="_blank">http://html5shim.googlecode.com/svn/trunk/html5.js</a>"></script><br clear="all">
<div>And one more thing - is it ok to put scripts directly in header.php in conditional IE comments (<font color="#444444" face="monospace"><span style="font-size:11px"><!--[if lt IE 9]> <![endif]--></span></font>)</div>
<div><br></div>Best regards,<br>Ola Łączek<br><div><br></div>
</div>
<br></div></div><div>_______________________________________________<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></div></blockquote></div><br></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></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>