From b607d81d50fe291fc5105431b63578ca8405376e Mon Sep 17 00:00:00 2001
From: Blake Erickson <o.blakeerickson@gmail.com>
Date: Thu, 12 Oct 2023 21:26:10 -0600
Subject: [PATCH] DEV: Change video placeholder click target (#23925)

Have the click target be the entire placeholder instead of just the play
button.
---
 .../app/instance-initializers/video-placeholder.js          | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/assets/javascripts/discourse/app/instance-initializers/video-placeholder.js b/app/assets/javascripts/discourse/app/instance-initializers/video-placeholder.js
index 17e3dc94f7d..c2788ade770 100644
--- a/app/assets/javascripts/discourse/app/instance-initializers/video-placeholder.js
+++ b/app/assets/javascripts/discourse/app/instance-initializers/video-placeholder.js
@@ -8,7 +8,7 @@ export default {
     withPluginApi("0.8.7", (api) => {
       function handleVideoPlaceholderClick(helper, event) {
         const parentDiv = event.target.closest(".video-placeholder-container");
-        const wrapper = event.target.closest(".video-placeholder-wrapper");
+        const wrapper = parentDiv.querySelector(".video-placeholder-wrapper");
 
         const videoHTML = `
         <video width="100%" height="100%" preload="metadata" controls style="display:none">
@@ -73,8 +73,8 @@ export default {
           container.appendChild(wrapper);
 
           overlay.classList.add("video-placeholder-overlay");
-          overlay.style.cursor = "pointer";
-          overlay.addEventListener(
+          container.style.cursor = "pointer";
+          container.addEventListener(
             "click",
             handleVideoPlaceholderClick.bind(null, helper),
             false