From 87a8fd3a362a5ae56e62f7aa7208102bb53d5707 Mon Sep 17 00:00:00 2001
From: nonmakina <nonmakina@leftypol.org>
Date: Tue, 26 Jan 2021 00:44:23 -0600
Subject: [PATCH] Adds support for embed button for youtube. Eliminates preview
 image fetched from youtube that was causing a leak

---
 inc/instance-config.php |  1 -
 js/youtube.js           | 33 ++++++++++++++++++++++-----------
 2 files changed, 22 insertions(+), 12 deletions(-)

diff --git a/inc/instance-config.php b/inc/instance-config.php
index 4b66f15f..b668571e 100644
--- a/inc/instance-config.php
+++ b/inc/instance-config.php
@@ -346,7 +346,6 @@ $config['enable_embedding'] = true;
 
 $config['youtube_js_html'] = '<div class="video-container" data-video="$2">'.
 '<a href="https://youtu.be/$2" target="_blank" class="file">'.
-'<img style="width:255px;height:190px;" src="//img.youtube.com/vi/$2/0.jpg" class="post-image"/>'.
 '</a></div>';
 
 $config['embedding'] = array();
diff --git a/js/youtube.js b/js/youtube.js
index 9fe81b60..4c4cb74a 100644
--- a/js/youtube.js
+++ b/js/youtube.js
@@ -25,21 +25,32 @@
 
 onready(function(){
 	var do_embed_yt = function(tag) {
-		$('div.video-container a', tag).click(function() {
-			var videoID = $(this.parentNode).data('video');
-		
-			$(this.parentNode).html('<iframe style="float:left;margin: 10px 20px" type="text/html" '+
-				'width="360" height="270" src="//www.youtube.com/embed/' + videoID +
-				'?autoplay=1&html5=1" allowfullscreen frameborder="0"/>');
+		const ON = "[Remove]";
+		const OFF = "[Embed]";
 
-			return false;
+		var videoNode = $('div.video-container', tag);
+		var videoId = videoNode.data('video');
+		var span = $("<span>[Embed]</span>");
+		var embedNode = $('<iframe style="float:left;margin: 10px 20px" type="text/html" '+
+				'width="360" height="270" src="//www.youtube.com/embed/' + videoId +
+				'?autoplay=1&html5=1" allowfullscreen frameborder="0"/>')
+		span.click(function() {
+			if (span.text() == ON){
+				videoNode.remove("iframe");
+				span.text(OFF);
+			} else{
+				videoNode.append(embedNode);
+				span.innerHTML = ON;
+			}
 		});
+
+		videoNode.append(span);
 	};
 	do_embed_yt(document);
 
-        // allow to work with auto-reload.js, etc.
-        $(document).on('new_post', function(e, post) {
-                do_embed_yt(post);
-        });
+	// allow to work with auto-reload.js, etc.
+	$(document).on('new_post', function(e, post) {
+			do_embed_yt(post);
+	});
 });