WordPress-C
/
WordPress
mirror of https://github.com/WordPress/WordPress.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

When dynamically instantiating `MediaElementPlayer` instances in JS, `preload="metadata"` is extremely wonky. `preload="none"` lets the file play right away. 

This has already been addressed for audio|video shortcodes, but also needs to be the case for playlists. When loading a playlist from a DOM cache or via AJAX, and when the sources match those already pre-loaded, the browser will sometimes donkey-punch the HTTP request for the first cached track selected.

Set `preload="none"` for the empty `<audio|video>` tag for playlists.

See commits related to #26779.


Built from https://develop.svn.wordpress.org/trunk@27974


git-svn-id: http://core.svn.wordpress.org/trunk@27804 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
Scott Taylor 2014-04-06 21:33:15 +00:00
parent 29b1f9a0de
commit 7689cc4a74
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
wp-includes/media.php
View File

@ -1269,7 +1269,7 @@ function wp_playlist_shortcode( $attr ) {
<?php if ( 'audio' === $type ): ?> <?php if ( 'audio' === $type ): ?>
<div class="wp-playlist-current-item"></div> <div class="wp-playlist-current-item"></div>
<?php endif ?> <?php endif ?>
<<?php echo $safe_type ?> controls="controls" preload="metadata" width="<?php <<?php echo $safe_type ?> controls="controls" preload="none" width="<?php
echo (int) $theme_width; echo (int) $theme_width;
?>"<?php if ( 'video' === $safe_type ): ?>"<?php if ( 'video' === $safe_type ):
echo ' height="', (int) $theme_height, '"'; echo ' height="', (int) $theme_height, '"';