From bf470704a5a3071cebb1d558efaef8542a16dde6 Mon Sep 17 00:00:00 2001
From: Thomas Lange <code@nerdmind.de>
Date: Tue, 26 Sep 2023 21:45:52 +0200
Subject: Add option to control preloading of video data

Fix #4110 by adding an option to control the preloading of video data on
page load. If disabled ("false"), the browser will not preload any video
data until the user explicitly hits the "Play" button.

If enabled ("true"), the default behavior will be used, which means the
browser decides how much of the video will be preloaded.
---
 assets/js/player.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'assets/js/player.js')

diff --git a/assets/js/player.js b/assets/js/player.js
index bb53ac24..398c66f8 100644
--- a/assets/js/player.js
+++ b/assets/js/player.js
@@ -3,7 +3,6 @@ var player_data = JSON.parse(document.getElementById('player_data').textContent)
 var video_data = JSON.parse(document.getElementById('video_data').textContent);
 
 var options = {
-    preload: 'auto',
     liveui: true,
     playbackRates: [0.25, 0.5, 0.75, 1.0, 1.25, 1.5, 1.75, 2.0],
     controlBar: {
@@ -35,6 +34,10 @@ if (player_data.aspect_ratio) {
     options.aspectRatio = player_data.aspect_ratio;
 }
 
+if (player_data.preload) {
+    options.preload = player_data.preload
+}
+
 var embed_url = new URL(location);
 embed_url.searchParams.delete('v');
 var short_url = location.origin + '/' + video_data.id + embed_url.search;
-- 
cgit v1.2.3


From 824cc1a5aa6321936357677e77b6638c96137225 Mon Sep 17 00:00:00 2001
From: Thomas Lange <code@nerdmind.de>
Date: Wed, 27 Sep 2023 15:25:29 +0200
Subject: Don't redefine the "preload" option in player.js

If the HTML5 "<video>" element defines the "preload" attribute directly,
it isn't necessary to redefine the "preload" option in the player.js.
---
 assets/js/player.js                       | 4 ----
 src/invidious/views/components/player.ecr | 3 +--
 2 files changed, 1 insertion(+), 6 deletions(-)

(limited to 'assets/js/player.js')

diff --git a/assets/js/player.js b/assets/js/player.js
index 398c66f8..7c920400 100644
--- a/assets/js/player.js
+++ b/assets/js/player.js
@@ -34,10 +34,6 @@ if (player_data.aspect_ratio) {
     options.aspectRatio = player_data.aspect_ratio;
 }
 
-if (player_data.preload) {
-    options.preload = player_data.preload
-}
-
 var embed_url = new URL(location);
 embed_url.searchParams.delete('v');
 var short_url = location.origin + '/' + video_data.id + embed_url.search;
diff --git a/src/invidious/views/components/player.ecr b/src/invidious/views/components/player.ecr
index f72fd9de..5c28358b 100644
--- a/src/invidious/views/components/player.ecr
+++ b/src/invidious/views/components/player.ecr
@@ -74,8 +74,7 @@
     "title" => video.title,
     "description" => HTML.escape(video.short_description),
     "thumbnail" => thumbnail,
-    "preferred_caption_found" => !preferred_captions.empty?,
-    "preload" => params.preload ? "auto" : "none"
+    "preferred_caption_found" => !preferred_captions.empty?
 }.to_pretty_json
 %>
 </script>
-- 
cgit v1.2.3