Introduce ability to hide shorts/livestreams in subscription feed

[config/config.example.yml]
- Introduce new default value for hide_shorts_and_live

[locales/en-US.json]
- Add translation label for hide_shorts_and_live

[src/invidious/config.cr]
- Add default value for hide_shorts_and_live to ConfigPreferences

[src/invidious/users.cr]
- Add filtering for shorts and listreams if hide_shorts_and_live is active

[src/invidious/routes/preferences.cr]
- Retrieve and convert hide_shorts_and_live option from preferences page
  to preferences object

[src/invidious/user/preferences.cr]
- Add hide_shorts_and_live property

[src/invidious/views/user/preferences.ecr]
- Add div, label and input for new hide_shorts_and_live option

config/config.example.yml

@@ -939,6 +939,14 @@ default_user_preferences:
   ##
   #sort: published
 
+  ##
+  ## In the "Subscription" feed, Only show the videos, no shorts
+  ## or livestreams.
+  ##
+  ## Accepted values: true, false
+  ## Default: false
+  ##
+  #hide_shorts_and_live: false
 
   # -----------------------------
   #  Miscellaneous

locales/en-US.json

@@ -133,6 +133,7 @@
     "Only show latest video from channel: ": "Only show latest video from channel: ",
     "Only show latest unwatched video from channel: ": "Only show latest unwatched video from channel: ",
     "preferences_unseen_only_label": "Only show unwatched: ",
+    "preferences_hide_shorts_and_live_label": "Hide shorts and live streams: ",
     "preferences_notifications_only_label": "Only show notifications (if there are any): ",
     "Enable web notifications": "Enable web notifications",
     "`x` uploaded a video": "`x` uploaded a video",

src/invidious/config.cr

@@ -47,6 +47,7 @@ struct ConfigPreferences
   property thin_mode : Bool = false
   property unseen_only : Bool = false
   property video_loop : Bool = false
+  property hide_shorts_and_live : Bool = false
   property extend_desc : Bool = false
   property volume : Int32 = 100
   property vr_mode : Bool = true

src/invidious/routes/preferences.cr

@@ -143,6 +143,10 @@ module Invidious::Routes::PreferencesRoute
     notifications_only ||= "off"
     notifications_only = notifications_only == "on"
 
+    hide_shorts_and_live = env.params.body["hide_shorts_and_live"]?.try &.as(String)
+    hide_shorts_and_live ||= "off"
+    hide_shorts_and_live = hide_shorts_and_live == "on"
+
     default_playlist = env.params.body["default_playlist"]?.try &.as(String)
 
     # Convert to JSON and back again to take advantage of converters used for compatibility
@@ -182,6 +186,7 @@ module Invidious::Routes::PreferencesRoute
       show_nick:                   show_nick,
       save_player_pos:             save_player_pos,
       default_playlist:            default_playlist,
+      hide_shorts_and_live:        hide_shorts_and_live,
     }.to_json)
 
     if user = env.get? "user"

src/invidious/user/preferences.cr

@@ -57,6 +57,7 @@ struct Preferences
   property volume : Int32 = CONFIG.default_user_preferences.volume
   property save_player_pos : Bool = CONFIG.default_user_preferences.save_player_pos
   property default_playlist : String? = nil
+  property hide_shorts_and_live : Bool = CONFIG.default_user_preferences.hide_shorts_and_live
 
   module BoolToString
     def self.to_json(value : String, json : JSON::Builder)

src/invidious/users.cr

@@ -83,6 +83,10 @@ def get_subscription_feed(user, max_results = 40, page = 1)
       end
     end
 
+    if user.preferences.hide_shorts_and_live
+      videos = videos.select { |v| v.length_seconds > 0 }
+    end
+
     case user.preferences.sort
     when "published - reverse"
       videos.sort_by!(&.published)

src/invidious/views/user/preferences.ecr

@@ -229,6 +229,11 @@
                     <input name="watch_history" id="watch_history" type="checkbox" <% if preferences.watch_history %>checked<% end %>>
                 </div>
 
+                <div class="pure-control-group">
+                    <label for="hide_shorts_and_live"><%= translate(locale, "preferences_hide_shorts_and_live_label") %></label>
+                    <input name="hide_shorts_and_live" id="hide_shorts_and_live" type="checkbox" <% if preferences.hide_shorts_and_live %>checked<% end %>>
+                </div>
+
                 <div class="pure-control-group">
                     <label for="annotations_subscribed"><%= translate(locale, "preferences_annotations_subscribed_label") %></label>
                     <input name="annotations_subscribed" id="annotations_subscribed" type="checkbox" <% if preferences.annotations_subscribed %>checked<% end %>>