From bde1df7a987e283eae2e40b94be5e515463fa41d Mon Sep 17 00:00:00 2001 From: Sam Potts Date: Tue, 17 Feb 2015 21:49:31 +1100 Subject: Fixing display issues in IE - Added class hooks for media type - Returning plyr instances when calling .setup() --- assets/js/plyr.js | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) (limited to 'assets/js/plyr.js') diff --git a/assets/js/plyr.js b/assets/js/plyr.js index c594b36c..76d2beba 100644 --- a/assets/js/plyr.js +++ b/assets/js/plyr.js @@ -41,7 +41,9 @@ seekTime: ".player-seek-time" }, classes: { - videoContainer: "player-video", + video: "player-video", + videoWrapper: "player-video-wrapper", + audio: "player-audio", stopped: "stopped", playing: "playing", muted: "muted", @@ -410,20 +412,23 @@ return false; } - // If there's no autoplay attribute, assume the video is stopped - _toggleClass(player.container, config.classes.stopped, (player.media.getAttribute("autoplay") === null)); - // Remove native video controls player.media.removeAttribute("controls"); - // Set type + // Set media type player.type = (player.media.tagName.toLowerCase() == "video" ? "video" : "audio"); + // Add type class + _toggleClass(player.container, config.classes[player.type], true); + + // If there's no autoplay attribute, assume the video is stopped and add state class + _toggleClass(player.container, config.classes.stopped, (player.media.getAttribute("autoplay") === null)); + // Inject the player wrapper if(player.type === "video") { // Create the wrapper div var wrapper = document.createElement("div"); - wrapper.setAttribute("class", config.classes.videoContainer); + wrapper.setAttribute("class", config.classes.videoWrapper); // Wrap the video in a container _wrap(player.media, wrapper); -- cgit v1.2.3