From: joshuaruesweg <ruesweg@woltlab.com>
Date: Mon, 22 Jun 2020 14:44:05 +0000 (+0200)
Subject: Fix broken HTML markup
X-Git-Tag: 5.2.7~11
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=6c03975f9848498ba098f5d2147fd68a070393ea;p=GitHub%2FWoltLab%2FWCF.git

Fix broken HTML markup
The video can be placed within normal text (inside `<p>` elements). `<div>` elements are not allowed within `<p>` elements. Because the class `.videoContainer` is already a block element (via CSS), we can easily change the element type to `<span>`, which is allowed inside of `<p>` elements.
---

diff --git a/com.woltlab.wcf/templates/__videoAttachmentBBCode.tpl b/com.woltlab.wcf/templates/__videoAttachmentBBCode.tpl
index da69480dbb..4343043bfb 100644
--- a/com.woltlab.wcf/templates/__videoAttachmentBBCode.tpl
+++ b/com.woltlab.wcf/templates/__videoAttachmentBBCode.tpl
@@ -1,6 +1,6 @@
-<div id="attachmentVideo_{@$attachmentIdentifier}" class="videoContainer" style="display: none;">
+<span id="attachmentVideo_{@$attachmentIdentifier}" class="videoContainer" style="display: none;">
 	<video src="{link controller='Attachment' object=$attachment}{/link}" controls></video>
-</div>
+</span>
 
 <a id="attachmentVideoLink_{@$attachmentIdentifier}" href="{link controller='Attachment' object=$attachment}{/link}">{$attachment->filename}</a>