[theme-reviewers] Enqueue scripts from CDN

[Srikanth Koneru]

I think option set to off by default should be okay.


On Wed, Aug 7, 2013 at 8:24 AM, Daniel <danielx386 at gmail.com> wrote:

> Something else, can a theme have the option to use a CDN but have it
> off by default, or is that a no no as well?
>
>
>
>
>
>
> On Wed, Aug 7, 2013 at 12:49 PM, Josh Pollock <jpollock412 at gmail.com>
> wrote:
> > @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.
> >
> >
> > On Tue, Aug 6, 2013 at 3:37 PM, Edward Caissie <edward.caissie at gmail.com
> >
> > wrote:
> >>
> >> 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.
> >>
> >> Edward Caissie
> >> aka Cais.
> >>
> >>
> >> On Tue, Aug 6, 2013 at 3:15 PM, Chip Bennett <chip at chipbennett.net>
> wrote:
> >>>
> >>> Everything the Theme calls that doesn't come from WordPress core *must*
> >>> be bundled with the Theme.
> >>>
> >>> Just about the only exception to that rule is an API call such as
> >>> GoogleFonts.
> >>>
> >>>
> >>> On Tue, Aug 6, 2013 at 2:37 PM, Ola Łączek <ola at bodera.com> wrote:
> >>>>
> >>>> Hello!
> >>>>
> >>>> Is it ok for theme author to enqueue scripts directly from CDN, and
> not
> >>>> include them with the theme?
> >>>> Example: <script
> >>>> src="http://html5shim.googlecode.com/svn/trunk/html5.js"></script>
> >>>> And one more thing - is it ok to put scripts directly in header.php in
> >>>> conditional IE comments (<!--[if lt IE 9]> <![endif]-->)
> >>>>
> >>>> Best regards,
> >>>> Ola Łączek
> >>>>
> >>>>
> >>>> _______________________________________________
> >>>> theme-reviewers mailing list
> >>>> theme-reviewers at lists.wordpress.org
> >>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
> >>>>
> >>>
> >>>
> >>> _______________________________________________
> >>> theme-reviewers mailing list
> >>> theme-reviewers at lists.wordpress.org
> >>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
> >>>
> >>
> >>
> >> _______________________________________________
> >> theme-reviewers mailing list
> >> theme-reviewers at lists.wordpress.org
> >> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
> >>
> >
> >
> > _______________________________________________
> > theme-reviewers mailing list
> > theme-reviewers at lists.wordpress.org
> > http://lists.wordpress.org/mailman/listinfo/theme-reviewers
> >
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130807/92e9d222/attachment.html>