PrivateCoffee/invidious
4
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

Don't redefine the "preload" option in player.js

Browse source 
If the HTML5 "<video>" element defines the "preload" attribute directly,
it isn't necessary to redefine the "preload" option in the player.js.
This commit is contained in:
Thomas Lange 2023-09-27 15:25:29 +02:00
parent bf470704a5
commit 824cc1a5aa
2 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
assets/js/player.js
View file

@ -34,10 +34,6 @@ if (player_data.aspect_ratio) {
options.aspectRatio = 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); var embed_url = new URL(location);
embed_url.searchParams.delete('v'); embed_url.searchParams.delete('v');
var short_url = location.origin + '/' + video_data.id + embed_url.search; var short_url = location.origin + '/' + video_data.id + embed_url.search;

3
src/invidious/views/components/player.ecr
View file

@ -74,8 +74,7 @@
"title" => video.title, "title" => video.title,
"description" => HTML.escape(video.short_description), "description" => HTML.escape(video.short_description),
"thumbnail" => thumbnail, "thumbnail" => thumbnail,
"preferred_caption_found" => !preferred_captions.empty?, "preferred_caption_found" => !preferred_captions.empty?
"preload" => params.preload ? "auto" : "none"
}.to_pretty_json }.to_pretty_json
%> %>
</script> </script>