From 141ecf221b553925a2266058c649c2262c2ee5aa Mon Sep 17 00:00:00 2001 From: Gary Katsevman Date: Mon, 22 Aug 2016 14:20:23 -0400 Subject: [PATCH] @gkatsev made videojs requirable in node. closes #3540 --- src/js/player.js | 2 +- src/js/utils/dom.js | 5 ++++- src/js/video.js | 4 +++- 3 files changed, 8 insertions(+), 3 deletions(-) diff --git a/src/js/player.js b/src/js/player.js index 0f58266d7..86053970c 100644 --- a/src/js/player.js +++ b/src/js/player.js @@ -2937,7 +2937,7 @@ Player.prototype.options_ = { 'textTrackSettings' ], - language: navigator.languages && navigator.languages[0] || navigator.userLanguage || navigator.language || 'en', + language: navigator && (navigator.languages && navigator.languages[0] || navigator.userLanguage || navigator.language) || 'en', // locales and their language translations languages: {}, diff --git a/src/js/utils/dom.js b/src/js/utils/dom.js index 66fe9f597..90f497303 100644 --- a/src/js/utils/dom.js +++ b/src/js/utils/dom.js @@ -67,7 +67,10 @@ function createQuerier(method) { if (isNonBlankString(context)) { context = document.querySelector(context); } - return (isEl(context) ? context : document)[method](selector); + + const ctx = isEl(context) ? context : document; + + return ctx[method] && ctx[method](selector); }; } diff --git a/src/js/video.js b/src/js/video.js index 92e358d16..28638cd15 100644 --- a/src/js/video.js +++ b/src/js/video.js @@ -108,7 +108,9 @@ if (window.VIDEOJS_NO_DYNAMIC_STYLE !== true) { style = stylesheet.createStyleElement('vjs-styles-defaults'); const head = Dom.$('head'); - head.insertBefore(style, head.firstChild); + if (head) { + head.insertBefore(style, head.firstChild); + } stylesheet.setTextContent(style, ` .video-js { width: 300px;