[theme-reviewers] help with review

Srikanth Koneru tskk79 at gmail.com
Tue Jul 2 19:25:46 UTC 2013

The line about server resources was unnecessary I guess, Since I wasn't
asking him to host it and it occurred to me a bit later that hosting the
video himself will be phoning home.

But if video is tiny and he really wants to embed it then bundling it and
using html5 video tag will negate the need for iframe. Pointing it out
purely for academic purposes.

On Wed, Jul 3, 2013 at 12:37 AM, Otto <otto at ottodestruct.com> wrote:

> Srikanth, that would make the theme needlessly large, and we have a
> size limit on themes in the repository. There's nothing wrong with
> including content from YouTube or Vimeo or some common video provider
> code and the like. These are commonplace and frequently used.
> The concern about including third party resources such as with iframes
> is primarily related to privacy and tracking, and also security. If
> you were including an iframe linking to your own site, then that would
> be a concern because you would be getting back information about who
> used your theme and on what sites. If you were including JS code from
> your site, then you could change that code and potentially "hack"
> other sites with it.
> Themes should load resources all from themselves and not from the
> theme author's sites. Including code from very widely used
> third-parties, where you're not receiving anything like tracking info
> and such, I would personally find okay.
> But I would question the wisdom of including this sort of thing as an
> embedded video on, say, a theme options page. Not everybody has super
> high speed connections, and mobile use is becoming a lot more
> prevalent. It might be best to hide the videos behind a normal clicked
> link, or to make a wholly separate page for them. In fact, you might
> consider not putting those video embeds in the theme at all, but
> instead having some kind of help link back to your own site for
> viewing them. This way, changing the videos and the help information
> and such doesn't actually require updating the theme every time. A
> normal link to your own site for help and information and other such
> things is perfectly acceptable.
> -Otto
> On Tue, Jul 2, 2013 at 1:40 PM, Srikanth Koneru <tskk79 at gmail.com> wrote:
> > @Greg, If your theme is html5, simply bundle the video(mp4, webm, ogg)
> along
> > with the theme, use the new video tag and for its source use  <?php echo
> > get_template_directory_uri(); ?>/location/video.mp4
> > That way you won't have to worry about burning your bandwidth and server
> > resources.
> _______________________________________________
> 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/20130703/8cd5eca5/attachment.html>

More information about the theme-reviewers mailing list