diff options
author | Sam Potts <sam@potts.es> | 2018-07-30 22:47:10 +1000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-07-30 22:47:10 +1000 |
commit | f964e34d8c01fe09ecbf99063be5f278cf8f3353 (patch) | |
tree | c693531622663e61e72d7cc486a1cada4c926fbc /src/js | |
parent | 96d371546c936afa0dcee0e5adca0ea3722a519f (diff) | |
parent | 021ba0b8e911184e06b3010cb00f66f5691aec0e (diff) | |
download | plyr-f964e34d8c01fe09ecbf99063be5f278cf8f3353.tar.lz plyr-f964e34d8c01fe09ecbf99063be5f278cf8f3353.tar.xz plyr-f964e34d8c01fe09ecbf99063be5f278cf8f3353.zip |
Merge pull request #1134 from mjfwebb/hide-empty-controls
Hide empty controls
Diffstat (limited to 'src/js')
-rw-r--r-- | src/js/controls.js | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/js/controls.js b/src/js/controls.js index 2bc8b4de..e95cfc86 100644 --- a/src/js/controls.js +++ b/src/js/controls.js @@ -1455,12 +1455,9 @@ const controls = { target = this.elements.container; } - // Inject controls HTML - if (is.element(container)) { - target.appendChild(container); - } else if (container) { - target.insertAdjacentHTML('beforeend', container); - } + // Inject controls HTML (needs to be before captions, hence "afterbegin") + const insertMethod = is.element(container) ? 'insertAdjacentElement' : 'insertAdjacentHTML'; + target[insertMethod]('afterbegin', container); // Find the elements if need be if (!is.element(this.elements.controls)) { |