MDL-70905 admin: Change default player dimensions to 360p.

NoelDeMartin · Apr 22, 2021 · af560d4 · af560d4
1 parent e29b256
commit af560d4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/admin/settings/plugins.php b/admin/settings/plugins.php
@@ -254,10 +254,10 @@
     $temp->add(new admin_setting_heading('managemediaplayerscommonheading', new lang_string('commonsettings', 'admin'), ''));
     $temp->add(new admin_setting_configtext('media_default_width',
         new lang_string('defaultwidth', 'core_media'), new lang_string('defaultwidthdesc', 'core_media'),
-        400, PARAM_INT, 10));
+        640, PARAM_INT, 10));
     $temp->add(new admin_setting_configtext('media_default_height',
         new lang_string('defaultheight', 'core_media'), new lang_string('defaultheightdesc', 'core_media'),
-        300, PARAM_INT, 10));
+        360, PARAM_INT, 10));
     $ADMIN->add('mediaplayers', $temp);
 
     // Convert plugins.

diff --git a/filter/mediaplugin/tests/filter_test.php b/filter/mediaplugin/tests/filter_test.php
@@ -93,7 +93,7 @@ function test_filter_mediaplugin_link() {
             '<a href="https://www.youtube.com/watch?v=uUhWl9Lm3OM">Valid link</a></pre><pre style="color: rgb(0, 0, 0); line-height: normal;">';
         $paddedurl = str_pad($originalurl, 6000, 'z');
         $validpaddedurl = '<p>Some text.</p><pre style="color: rgb(0, 0, 0); line-height: normal;"><span class="mediaplugin mediaplugin_youtube">
-<iframe title="Valid link" width="400" height="300"
+<iframe title="Valid link" width="640" height="360"
   src="https://www.youtube.com/embed/uUhWl9Lm3OM?rel=0&amp;wmode=transparent" frameborder="0" allowfullscreen="1"></iframe>
 </span></pre><pre style="color: rgb(0, 0, 0); line-height: normal;">';
         $validpaddedurl = str_pad($validpaddedurl, 6000 + (strlen($validpaddedurl) - strlen($originalurl)), 'z');