updates #1795: Only download metadata for video files

c3102561 · JC Brand · c6c65749 · c3102561 · c3102561 · c3102561
Commit c3102561 authored Dec 18, 2019 by JC Brand
Show whitespace changes
Inline Side-by-side

Showing with 4 additions and 11 deletions

spec/messages.js spec/messages.js +2 -4

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

src/utils/html.js src/utils/html.js +1 -5

No files found.
--- a/spec/messages.js
+++ b/spec/messages.js
@@ -1973,8 +1973,7 @@
                let media = view.el.querySelector('.chat-msg .chat-msg__media');
                expect(media.innerHTML.replace(/(\r\n|\n|\r)/gm, "")).toEqual(
                    '<!-- src/templates/video.html -->'+
-                    '<video controls="" src="https://montague.lit/video.mp4" style="max-height: 50vh"></video>'+
+                    '<video controls="" preload="metadata" src="https://montague.lit/video.mp4" style="max-height: 50vh"></video>');
-                    '<a target="_blank" rel="noopener" href="https://montague.lit/video.mp4">Download video file "video.mp4"</a>');
                // If the <url> and <body> contents is the same, don't duplicate.
                stanza = u.toStanza(`
@@ -1991,8 +1990,7 @@
                media = view.el.querySelector('.chat-msg:last-child .chat-msg__media');
                expect(media.innerHTML.replace(/(\r\n|\n|\r)/gm, "")).toEqual(
                    '<!-- src/templates/video.html -->'+
-                    '<video controls="" src="https://montague.lit/video.mp4" style="max-height: 50vh"></video>'+
+                    '<video controls="" preload="metadata" src="https://montague.lit/video.mp4" style="max-height: 50vh"></video>');
-                    '<a target="_blank" rel="noopener" href="https://montague.lit/video.mp4">Download video file "video.mp4"</a>');
                done();
            }));

--- a/src/templates/video.html
+++ b/src/templates/video.html
-<video controls src="{{{o.url}}}" style="max-height: 50vh"></video>
+<video controls preload="metadata" src="{{{o.url}}}" style="max-height: 50vh"></video>
-<a target="_blank" rel="noopener" href="{{{o.url}}}">{{{o.label_download}}}</a>
--- a/src/utils/html.js
+++ b/src/utils/html.js
@@ -223,11 +223,7 @@ u.renderImageURLs = function (_converse, el) {
 u.renderMovieURL = function (_converse, url) {
    const uri = new URI(url);
    if (u.isVideoURL(uri)) {
-        const { __ } = _converse;
+        return tpl_video({url});
-        return tpl_video({
-            'url': url,
-            'label_download': __('Download video file "%1$s"', decodeURI(uri.filename()))
-        })
    }
    return url;
 };