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

Merge pull request #1592 from saltycrys/remove-top-feed-option

Browse source 
Remove "Top" feed option from preferences
This commit is contained in:
TheFrenchGhosty 2020-12-19 19:56:11 +00:00 committed by GitHub
commit dba7dc4e96
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/invidious/routes/user_preferences.cr
View file

@ -74,7 +74,7 @@ class Invidious::Routes::UserPreferences < Invidious::Routes::BaseRoute
default_home = env.params.body["default_home"]?.try &.as(String) || CONFIG.default_user_preferences.default_home default_home = env.params.body["default_home"]?.try &.as(String) || CONFIG.default_user_preferences.default_home
feed_menu = [] of String feed_menu = [] of String
5.times do |index| 4.times do |index|
option = env.params.body["feed_menu[#{index}]"]?.try &.as(String) || "" option = env.params.body["feed_menu[#{index}]"]?.try &.as(String) || ""
if !option.empty? if !option.empty?
feed_menu << option feed_menu << option
@ -146,7 +146,7 @@ class Invidious::Routes::UserPreferences < Invidious::Routes::BaseRoute
config.default_user_preferences.default_home = env.params.body["admin_default_home"]?.try &.as(String) || config.default_user_preferences.default_home config.default_user_preferences.default_home = env.params.body["admin_default_home"]?.try &.as(String) || config.default_user_preferences.default_home
admin_feed_menu = [] of String admin_feed_menu = [] of String
5.times do |index| 4.times do |index|
option = env.params.body["admin_feed_menu[#{index}]"]?.try &.as(String) || "" option = env.params.body["admin_feed_menu[#{index}]"]?.try &.as(String) || ""
if !option.empty? if !option.empty?
admin_feed_menu << option admin_feed_menu << option

4
src/invidious/views/preferences.ecr
View file

@ -130,9 +130,9 @@
</div> </div>
<% if env.get?("user") %> <% if env.get?("user") %>
<% feed_options = {"", "Popular", "Top", "Trending", "Subscriptions", "Playlists"} %> <% feed_options = {"", "Popular", "Trending", "Subscriptions", "Playlists"} %>
<% else %> <% else %>
<% feed_options = {"", "Popular", "Top", "Trending"} %> <% feed_options = {"", "Popular", "Trending"} %>
<% end %> <% end %>
<div class="pure-control-group"> <div class="pure-control-group">