yetangitu/Peertube
1
0
Fork 0
mirror of https://github.com/Chocobozzz/PeerTube.git synced 2025-10-06 03:50:26 +02:00
Code Issues Releases Wiki Activity

Styling

Browse source 
Also prefer "0" instead of "none" because outerHTML converts the later
to "medium" (I don't know why)
This commit is contained in:
Chocobozzz 2025-06-27 11:01:18 +02:00
parent 70010fac73
commit 532020e2af
No known key found for this signature in database
GPG key ID: 583A612D890159BE
3 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

2
client/src/root-helpers/video.ts
View file

@ -14,7 +14,7 @@ export function buildVideoOrPlaylistEmbed (options: {
iframe.width = responsive ? '100%' : '560'
iframe.height = responsive ? '100%' : '315'
iframe.src = embedUrl
iframe.style.border = 'none'
iframe.style.border = '0'
iframe.allow = 'fullscreen'
iframe.sandbox.add('allow-same-origin', 'allow-scripts', 'allow-popups', 'allow-forms')

8
packages/tests/src/api/server/services.ts
View file

@ -80,7 +80,7 @@ describe('Test services', function () {
const res = await server.services.getOEmbed({ oembedUrl })
const expectedHtml = '<iframe width="560" height="315" sandbox="allow-same-origin allow-scripts allow-popups allow-forms" ' +
`title="${video.name}" src="http://${server.host}/videos/embed/${video.shortUUID}${suffix.output}" ` +
'style="border:none" allow="fullscreen"></iframe>'
'style="border: none" allow="fullscreen"></iframe>'
const expectedThumbnailUrl = 'http://' + server.host + video.previewPath
@ -104,7 +104,7 @@ describe('Test services', function () {
const res = await server.services.getOEmbed({ oembedUrl })
const expectedHtml = '<iframe width="560" height="315" sandbox="allow-same-origin allow-scripts allow-popups allow-forms" ' +
`title="${playlistDisplayName}" src="http://${server.host}/video-playlists/embed/${playlistShortUUID}${suffix.output}" ` +
'style="border:none" allow="fullscreen"></iframe>'
'style="border: none" allow="fullscreen"></iframe>'
expect(res.body.html).to.equal(expectedHtml)
expect(res.body.title).to.equal('The Life and Times of Scrooge McDuck')
@ -125,7 +125,7 @@ describe('Test services', function () {
const expectedHtml = '<iframe width="560" height="315" sandbox="allow-same-origin allow-scripts allow-popups allow-forms" ' +
`title="${video.name}" src="http://${server.host}/videos/embed/${video.shortUUID}${query}" ` +
'style="border:none" allow="fullscreen"></iframe>'
'style="border: none" allow="fullscreen"></iframe>'
expect(res.body.html).to.equal(expectedHtml)
})
@ -140,7 +140,7 @@ describe('Test services', function () {
const res = await server.services.getOEmbed({ oembedUrl, format, maxHeight, maxWidth })
const expectedHtml = '<iframe width="50" height="50" sandbox="allow-same-origin allow-scripts allow-popups allow-forms" ' +
`title="${video.name}" src="http://${server.host}/videos/embed/${video.shortUUID}" ` +
'style="border:none" allow="fullscreen"></iframe>'
'style="border: none" allow="fullscreen"></iframe>'
expect(res.body.html).to.equal(expectedHtml)
expect(res.body.title).to.equal(video.name)

2
server/core/controllers/services.ts
View file

@ -137,7 +137,7 @@ function buildOEmbed (options: {
}
const html = `<iframe width="${embedWidth}" height="${embedHeight}" sandbox="allow-same-origin allow-scripts allow-popups allow-forms" ` +
`title="${embedTitle}" src="${embedUrl}" style="border:none" allow="fullscreen"></iframe>`
`title="${embedTitle}" src="${embedUrl}" style="border: none" allow="fullscreen"></iframe>`
const json: any = {
type: 'video',