Set max-height for embedded videos.

Also udpate markup to not use `<source>`. updates #1189

Set max-height for embedded videos.
Also udpate markup to not use `<source>`. updates #1189
af35d461 · JC Brand · b9c6a29f · af35d461 · af35d461 · af35d461
Commit af35d461 authored Sep 24, 2018 by JC Brand
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 3 deletions

dist/converse.js dist/converse.js +2 -2

fullscreen.html fullscreen.html +1 -0

src/templates/video.html src/templates/video.html +1 -1

No files found.
--- a/dist/converse.js
+++ b/dist/converse.js
@@ -81419,9 +81419,9 @@ return __p
 var _ = {escape:__webpack_require__(/*! ./node_modules/lodash/escape.js */ "./node_modules/lodash/escape.js")};
 module.exports = function(o) {
 var __t, __p = '', __e = _.escape;
-__p += '<!-- src/templates/video.html -->\n<video controls><source src="' +
+__p += '<!-- src/templates/video.html -->\n<video controls src="' +
 __e(o.url) +
-'" type="video/mp4"></video>\n<a target="_blank" rel="noopener" href="' +
+'" type="video/mp4" style="max-height: 50vh"></video>\n<a target="_blank" rel="noopener" href="' +
 __e(o.url) +
 '">' +
 __e(o.label_download) +
--- a/fullscreen.html
+++ b/fullscreen.html
@@ -9,6 +9,7 @@
    <noscript><p><img src="//stats.opkode.com/piwik.php?idsite=5" style="border:0;" alt="" /></p></noscript>
    <link type="text/css" rel="stylesheet" media="screen" href="css/converse.css" />
    <link type="text/css" rel="stylesheet" media="screen" href="css/fullpage.css" />
+    <script src="3rdparty/libsignal-protocol.js"></script>
    <script src="dist/converse.js"></script>
 </head>
 <body class="reset">

--- a/src/templates/video.html
+++ b/src/templates/video.html
-<video controls><source src="{{{o.url}}}" type="video/mp4"></video>
+<video controls src="{{{o.url}}}" type="video/mp4" style="max-height: 50vh"></video>
 <a target="_blank" rel="noopener" href="{{{o.url}}}">{{{o.label_download}}}</a>