Merge pull request #3159 from getnikola/fix-3150

Fix #3150
getnikola · Sep 7, 2018 · 473f7b6 · 473f7b6 · python-friends · Sep 7, 2018
2 parents 29fab6d + 3d03158
commit 473f7b6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -57,6 +57,7 @@ Features
 Bugfixes
 --------
 
+* Pass arguments to youtube directive unchanged (Issue #3150)
 * Fix listing installed themes if theme directory is missing.
 * Watch correct output folder in ``nikola auto`` (Issue #3119)
 * Fix post fragment dependencies when posts are only available in a

diff --git a/nikola/plugins/compile/rest/youtube.py b/nikola/plugins/compile/rest/youtube.py
@@ -67,8 +67,8 @@ class Youtube(Directive):
     has_content = True
     required_arguments = 1
     option_spec = {
-        "width": directives.positive_int,
-        "height": directives.positive_int,
+        "width": directives.unchanged,
+        "height": directives.unchanged,
         "align": _align_choice
     }
 
@@ -80,7 +80,7 @@ def run(self):
             'width': 560,
             'height': 315,
         }
-        options.update(self.options)
+        options.update({k: v for k, v in self.options.items() if v})
         if self.options.get('align') in _align_options_base:
             options['align'] = ' align-' + self.options['align']
         else: