[theme-reviewers] more experienced feedback
edward.caissie at gmail.com
Sat Dec 24 15:56:39 UTC 2011
@Kirk - Leave a comment to the effect of what Otto has suggested why this
method should not be used.
@Otto et al. - This should go into the Guidelines as *not* "best practice",
again for the same reason. "Banning" seems like such a harsh word to use,
I'm thinking simply not-approve-able due to the potential for "common"
end-user conditions to cause the theme to simply not work, i.e.: intranet
I'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.
The question is now where to put that into the guidelines as I recommend it
become effective with the "new" 3.3 guidelines. Any one have any
On Fri, Dec 23, 2011 at 7:42 PM, Otto <otto at ottodestruct.com> wrote:
> On Fri, Dec 23, 2011 at 4:32 PM, Kirk Wight <kwight at kwight.ca> wrote:
> > In particular, the theme uses a bunch of external APIs for the post
> > I've searched the mailing list archives, and didn't find anything
> > banning it, but, well, it weirds me out. A look at how the options are
> > implemented would be appreciated too.
> While there's nothing wrong with the files he's including and the
> sources are trustworthy enough, these files should be included in the
> theme and not added directly from external sites.
> In particular:
> wp_enqueue_style( 'html5reset');
> 'http://nje.github.com/jquery-tmpl/jquery.tmpl.js', array('jquery'),
> This is bad because it adds a dependency on those sites. This means
> the theme won't work for some cases, such as on an internal intranet
> where the user viewing the site has no access to the public internet
> (this is a *far* more common use case than you might think).
> While there's no explicit guidelines prohibiting it that I found in
> the theme review list, I kinda sorta think that it should be banned.
> There's no good reason a theme can't simply package up these libraries
> in the theme, presuming the licenses on the libraries is compatible.
> YUI is BSD licensed, JQuery Template is MIT or GPLv2, and the reset
> CSS is public domain. All compatible.
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the theme-reviewers