Skip to content

Commit afb052c

Browse files
Optimize video tag
1 parent 6a64215 commit afb052c

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

scripts/tags/video.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,5 @@
55
'use strict';
66

77
module.exports = function(args) {
8-
return `<video src="${args}" preload="metadata" controls playsinline poster="">Sorry, your browser does not support the video tag.</video>`;
8+
return `<video src="${args[0]}" preload="metadata" controlslist="nodownload" controls playsinline poster="${args[1] || ''}"></video>`;
99
};

test/tags/video.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@ describe('video', () => {
66
const postVideo = require('../../scripts/tags/video');
77

88
it('default', () => {
9-
postVideo(['https://example.com/sample.mp4']).should.eql('<video src="https://example.com/sample.mp4" preload="metadata" controls playsinline poster="">Sorry, your browser does not support the video tag.</video>');
9+
postVideo(['https://example.com/sample.mp4']).should.eql('<video src="https://example.com/sample.mp4" preload="metadata" controlslist="nodownload" controls playsinline poster=""></video>');
10+
});
11+
12+
it('poster', () => {
13+
postVideo(['https://example.com/sample.mp4', 'https://example.com/sample.jpg']).should.eql('<video src="https://example.com/sample.mp4" preload="metadata" controlslist="nodownload" controls playsinline poster="https://example.com/sample.jpg"></video>');
1014
});
1115
});

0 commit comments

Comments
 (0)