From 1c7c0a84264b9a896b4b57c9dc6e583cbde74391 Mon Sep 17 00:00:00 2001 From: Ryan Mohr Date: Mon, 10 Mar 2014 15:48:38 -1000 Subject: [PATCH] Apparently YouTube uses 16x9 now too. --- lib/widgets/youtube/youtube.js | 8 ++++---- test/widgets/youtube/youtube.html | 2 +- test/widgets/youtube/youtube.js | 4 ++-- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/widgets/youtube/youtube.js b/lib/widgets/youtube/youtube.js index 16c2c9c..f774ffb 100644 --- a/lib/widgets/youtube/youtube.js +++ b/lib/widgets/youtube/youtube.js @@ -8,14 +8,14 @@ // Options: // // ``` -// aspect desired aspect ratio [sd], hd +// aspect desired aspect ratio [hd], sd // ``` // // Examples: // // ``` -// Include video with id 1234 at 16x9 with autoplay -// [[youtube/1234?aspect=16x9&autoplay=1]] +// Include video with id 1234 in standard def with autoplay +// [[youtube/1234?aspect=sd&autoplay=1]] // ``` // function render(id, options) { @@ -26,5 +26,5 @@ function render(id, options) { } module.exports = function(widgets) { - widgets.add("youtube/:id", render, {aspect: "sd"}); + widgets.add("youtube/:id", render, {aspect: "hd"}); }; diff --git a/test/widgets/youtube/youtube.html b/test/widgets/youtube/youtube.html index cea3ee4..a79c68d 100644 --- a/test/widgets/youtube/youtube.html +++ b/test/widgets/youtube/youtube.html @@ -1,4 +1,4 @@ -
+
diff --git a/test/widgets/youtube/youtube.js b/test/widgets/youtube/youtube.js index 65c4d20..6783489 100644 --- a/test/widgets/youtube/youtube.js +++ b/test/widgets/youtube/youtube.js @@ -10,11 +10,11 @@ describe("Youtube", function() { describe("aspect", function() { it("default [sd]", function() { - expect(render("youtube/123")).to.contain('data-aspect-ratio="sd"'); + expect(render("youtube/123")).to.contain('data-aspect-ratio="hd"'); }); it("custom", function() { - expect(render("youtube/123?aspect=hd")).to.contain('data-aspect-ratio="hd"'); + expect(render("youtube/123?aspect=other")).to.contain('data-aspect-ratio="other"'); }); }); });