From 6fe50ddd1e02e06e755fb8ef4eb0fe6086a97402 Mon Sep 17 00:00:00 2001
From: intriguedlife <39467654+intriguedlife@users.noreply.github.com>
Date: Sat, 8 Feb 2025 20:32:27 +0100
Subject: [PATCH 1/5] [xExtension-YouTube] add player autosize feature

---
 xExtension-YouTube/configure.phtml  |  7 +++++++
 xExtension-YouTube/extension.php    | 29 ++++++++++++++++++++++++++++-
 xExtension-YouTube/i18n/de/ext.php  |  1 +
 xExtension-YouTube/i18n/en/ext.php  |  1 +
 xExtension-YouTube/i18n/fr/ext.php  |  1 +
 xExtension-YouTube/i18n/tr/ext.php  |  1 +
 xExtension-YouTube/static/style.css |  5 +++++
 7 files changed, 44 insertions(+), 1 deletion(-)
 create mode 100644 xExtension-YouTube/static/style.css

diff --git a/xExtension-YouTube/configure.phtml b/xExtension-YouTube/configure.phtml
index 8828fe7d..8895abc7 100644
--- a/xExtension-YouTube/configure.phtml
+++ b/xExtension-YouTube/configure.phtml
@@ -6,6 +6,13 @@ declare(strict_types=1);
 	<input type="hidden" name="_csrf" value="<?php echo FreshRSS_Auth::csrfToken(); ?>" />
 	<div class="form-group">
 
+		<div class="group-controls">
+			<label class="checkbox" for="yt_autosize">
+				<input type="checkbox" id="yt_autosize" name="yt_autosize" value="1" <?php echo $this->isAutoSize() ? 'checked' : ''; ?>>
+				<?php echo _t('ext.yt_videos.autosize'); ?>
+			</label>
+		</div>
+
 		<label class="group-name" for="yt_height"><?php echo _t('ext.yt_videos.height'); ?></label>
 		<div class="group-controls">
 			<input type="number" id="yt_height" name="yt_height" value="<?php echo $this->getHeight(); ?>" min="50" data-leave-validation="1">
diff --git a/xExtension-YouTube/extension.php b/xExtension-YouTube/extension.php
index c8bec521..400b0918 100644
--- a/xExtension-YouTube/extension.php
+++ b/xExtension-YouTube/extension.php
@@ -9,6 +9,10 @@
  */
 final class YouTubeExtension extends Minz_Extension
 {
+	/**
+	 * Whether we set the Youtube iframe to autosize
+	 */
+	private bool $autoSize = false;
 	/**
 	 * Video player width
 	 */
@@ -32,6 +36,8 @@ final class YouTubeExtension extends Minz_Extension
 	#[\Override]
 	public function init(): void
 	{
+		Minz_View::appendStyle($this->getFileUrl('style.css', 'css'));
+	
 		$this->registerHook('entry_before_display', [$this, 'embedYouTubeVideo']);
 		$this->registerHook('check_url_before_add', [self::class, 'convertYoutubeFeedUrl']);
 		$this->registerTranslates();
@@ -63,6 +69,11 @@ public function loadConfigValues(): void
 			return;
 		}
 
+		$autoSize = FreshRSS_Context::userConf()->attributeBool('yt_autosize');
+		if ($autoSize !== null) {
+			$this->autoSize = $autoSize;
+		}
+
 		$width = FreshRSS_Context::userConf()->attributeInt('yt_player_width');
 		if ($width !== null) {
 			$this->width = $width;
@@ -84,6 +95,15 @@ public function loadConfigValues(): void
 		}
 	}
 
+	/**
+	 * Returns whether this extension enables autosize for the YouTube player iframe.
+	 * You have to call loadConfigValues() before this one, otherwise you get default values.
+	 */
+	public function isAutoSize(): bool
+	{
+		return $this->autoSize;
+	}
+
 	/**
 	 * Returns the width in pixel for the YouTube player iframe.
 	 * You have to call loadConfigValues() before this one, otherwise you get default values.
@@ -181,7 +201,13 @@ public function getHtml(FreshRSS_Entry $entry, string $url): string
 	{
 		$content = '';
 
-		$iframe = '<iframe class="youtube-plugin-video"
+		if ($this->autoSize) {
+			$class = "yt_autosize_true";
+		} else {
+			$class = "yt_autosize_false";
+		}
+
+		$iframe = '<iframe class="' . $class . '"
 				style="height: ' . $this->height . 'px; width: ' . $this->width . 'px;"
 				width="' . $this->width . '"
 				height="' . $this->height . '"
@@ -252,6 +278,7 @@ public function handleConfigureAction(): void
 		$this->registerTranslates();
 
 		if (Minz_Request::isPost()) {
+			FreshRSS_Context::userConf()->_attribute('yt_autosize', Minz_Request::paramBoolean('yt_autosize'));
 			FreshRSS_Context::userConf()->_attribute('yt_player_height', Minz_Request::paramInt('yt_height'));
 			FreshRSS_Context::userConf()->_attribute('yt_player_width', Minz_Request::paramInt('yt_width'));
 			FreshRSS_Context::userConf()->_attribute('yt_show_content', Minz_Request::paramBoolean('yt_show_content'));
diff --git a/xExtension-YouTube/i18n/de/ext.php b/xExtension-YouTube/i18n/de/ext.php
index e6926707..35ca4caa 100644
--- a/xExtension-YouTube/i18n/de/ext.php
+++ b/xExtension-YouTube/i18n/de/ext.php
@@ -2,6 +2,7 @@
 
 return array(
 	'yt_videos' => array(
+		'autosize' => 'Autogröße des Players',
 		'height' => 'Höhe des Players',
 		'width' => 'Breite des Players',
 		'updates' => 'Die neueste Version des Plugins findest Du bei',
diff --git a/xExtension-YouTube/i18n/en/ext.php b/xExtension-YouTube/i18n/en/ext.php
index 6f37c024..234fc36a 100644
--- a/xExtension-YouTube/i18n/en/ext.php
+++ b/xExtension-YouTube/i18n/en/ext.php
@@ -2,6 +2,7 @@
 
 return array(
 	'yt_videos' => array(
+		'autosize' => 'Player autosize',
 		'height' => 'Player height',
 		'width' => 'Player width',
 		'updates' => 'You can find the latest extension version at',
diff --git a/xExtension-YouTube/i18n/fr/ext.php b/xExtension-YouTube/i18n/fr/ext.php
index 3beee1b8..50e8f57a 100644
--- a/xExtension-YouTube/i18n/fr/ext.php
+++ b/xExtension-YouTube/i18n/fr/ext.php
@@ -2,6 +2,7 @@
 
 return array(
 	'yt_videos' => array(
+		'autosize' => 'Taille automatique du lecteur',
 		'height' => 'Hauteur du lecteur',
 		'width' => 'Largeur du lecteur',
 		'updates' => 'Vous pouvez trouver la dernière mise à jour de l’extension sur ',
diff --git a/xExtension-YouTube/i18n/tr/ext.php b/xExtension-YouTube/i18n/tr/ext.php
index a206d2fc..4681ec81 100644
--- a/xExtension-YouTube/i18n/tr/ext.php
+++ b/xExtension-YouTube/i18n/tr/ext.php
@@ -2,6 +2,7 @@
 
 return array(
 	'yt_videos' => array(
+		'autosize' => 'Oynatıcı otomatik boyutlandırma',
 		'height' => 'Oynatıcı yükseklik',
 		'width' => 'Oynatıcı genişlik',
 		'updates' => 'En son uzantı sürümünü şu adreste bulabilirsiniz:',
diff --git a/xExtension-YouTube/static/style.css b/xExtension-YouTube/static/style.css
new file mode 100644
index 00000000..bec82a83
--- /dev/null
+++ b/xExtension-YouTube/static/style.css
@@ -0,0 +1,5 @@
+.yt_autosize_true {
+    height: 100%;
+    width:  100%;
+    aspect-ratio: 16 / 9;
+}
\ No newline at end of file

From 8ed35c8f761dd60873c50cb4ebe22d4d1291a31b Mon Sep 17 00:00:00 2001
From: intriguedlife <39467654+intriguedlife@users.noreply.github.com>
Date: Sat, 8 Feb 2025 21:50:40 +0100
Subject: [PATCH 2/5] [xExtension-YouTube] correct player autosize variable
 name

---
 xExtension-YouTube/extension.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/xExtension-YouTube/extension.php b/xExtension-YouTube/extension.php
index 400b0918..85f0c21e 100644
--- a/xExtension-YouTube/extension.php
+++ b/xExtension-YouTube/extension.php
@@ -69,7 +69,7 @@ public function loadConfigValues(): void
 			return;
 		}
 
-		$autoSize = FreshRSS_Context::userConf()->attributeBool('yt_autosize');
+		$autoSize = FreshRSS_Context::userConf()->attributeBool('yt_player_autosize');
 		if ($autoSize !== null) {
 			$this->autoSize = $autoSize;
 		}
@@ -278,7 +278,7 @@ public function handleConfigureAction(): void
 		$this->registerTranslates();
 
 		if (Minz_Request::isPost()) {
-			FreshRSS_Context::userConf()->_attribute('yt_autosize', Minz_Request::paramBoolean('yt_autosize'));
+			FreshRSS_Context::userConf()->_attribute('yt_player_autosize', Minz_Request::paramBoolean('yt_autosize'));
 			FreshRSS_Context::userConf()->_attribute('yt_player_height', Minz_Request::paramInt('yt_height'));
 			FreshRSS_Context::userConf()->_attribute('yt_player_width', Minz_Request::paramInt('yt_width'));
 			FreshRSS_Context::userConf()->_attribute('yt_show_content', Minz_Request::paramBoolean('yt_show_content'));

From 284445ed727340d1b77a3d881b004dba19c3fb5e Mon Sep 17 00:00:00 2001
From: intriguedlife <39467654+intriguedlife@users.noreply.github.com>
Date: Sat, 8 Feb 2025 23:06:44 +0100
Subject: [PATCH 3/5] [xExtension-YouTube] Correct CSS formatting

---
 xExtension-YouTube/static/style.css | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/xExtension-YouTube/static/style.css b/xExtension-YouTube/static/style.css
index bec82a83..32c07c8a 100644
--- a/xExtension-YouTube/static/style.css
+++ b/xExtension-YouTube/static/style.css
@@ -1,5 +1,5 @@
 .yt_autosize_true {
-    height: 100%;
-    width:  100%;
-    aspect-ratio: 16 / 9;
-}
\ No newline at end of file
+	width: 100%;
+	height: 100%;
+	aspect-ratio: 16 / 9;
+}

From bb097e52a76eac96507d57cc25db0eeea9e294e4 Mon Sep 17 00:00:00 2001
From: intriguedlife <39467654+intriguedlife@users.noreply.github.com>
Date: Sun, 9 Feb 2025 00:12:07 +0100
Subject: [PATCH 4/5] [xExtension-YouTube] refine iframe element logic

---
 xExtension-YouTube/extension.php    | 11 ++++-------
 xExtension-YouTube/static/style.css |  3 +--
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/xExtension-YouTube/extension.php b/xExtension-YouTube/extension.php
index 85f0c21e..732cce73 100644
--- a/xExtension-YouTube/extension.php
+++ b/xExtension-YouTube/extension.php
@@ -200,17 +200,14 @@ public function getHtmlPeerTubeContentForLink(FreshRSS_Entry $entry, string $lin
 	public function getHtml(FreshRSS_Entry $entry, string $url): string
 	{
 		$content = '';
-
+	
 		if ($this->autoSize) {
-			$class = "yt_autosize_true";
+			$iframe_attribute = 'class="yt_player_autosize"';
 		} else {
-			$class = "yt_autosize_false";
+			$iframe_attribute = 'width="' . $this->width . '" height="' . $this->height . '"';
 		}
 
-		$iframe = '<iframe class="' . $class . '"
-				style="height: ' . $this->height . 'px; width: ' . $this->width . 'px;"
-				width="' . $this->width . '"
-				height="' . $this->height . '"
+		$iframe = '<iframe ' . $iframe_attribute . '
 				src="' . $url . '"
 				frameborder="0"
 				allowFullScreen></iframe>';
diff --git a/xExtension-YouTube/static/style.css b/xExtension-YouTube/static/style.css
index bec82a83..3caea388 100644
--- a/xExtension-YouTube/static/style.css
+++ b/xExtension-YouTube/static/style.css
@@ -1,5 +1,4 @@
-.yt_autosize_true {
-    height: 100%;
+.yt_player_autosize {
     width:  100%;
     aspect-ratio: 16 / 9;
 }
\ No newline at end of file

From 94a9396b43cc0c70c8e1f8f6c4341d20ca12964c Mon Sep 17 00:00:00 2001
From: intriguedlife <39467654+intriguedlife@users.noreply.github.com>
Date: Sun, 9 Feb 2025 00:23:09 +0100
Subject: [PATCH 5/5] [xExtension-YouTube] CSS correction 3

---
 xExtension-YouTube/static/style.css | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xExtension-YouTube/static/style.css b/xExtension-YouTube/static/style.css
index ec5312a3..aaf1420a 100644
--- a/xExtension-YouTube/static/style.css
+++ b/xExtension-YouTube/static/style.css
@@ -1,4 +1,4 @@
-.yt_autosize_true {
+.yt_player_autosize {
 	width: 100%;
 	aspect-ratio: 16 / 9;
 }
\ No newline at end of file