Eisa01 / mpv-scripts

This repository contain scripts I have made for mpv media player...
BSD 2-Clause "Simplified" License
507 stars 35 forks source link

add main_list options, to change main list #31

Closed dyphire closed 2 years ago

Eisa01 commented 2 years ago

Nice idea! didn't think of this option.

I'll review and test, hopefully I didn't use display_list for some internal stuff (maybe I'm overthinking).

Though what's the difference if you change the main list instead of using a filter keybind?

Eisa01 commented 2 years ago

Actually this gave me a nice idea of a better approach to define keybinds and filter? What do you think if I convert this option to a json table that accepts two parameters, e.g.:

open_list_keybinds=[[
[ ["b", "all",], ["B", 'all'], ["k", "keybinds"], ["K", "keybinds"] ]
]]

This will also allow me to get rid of the specific options I am creating for each filter

dyphire commented 2 years ago

Actually this gave me a nice idea of a better approach to define keybinds and filter? What do you think if I convert this option to a json table that accepts two parameters, e.g.:

It looks better

Eisa01 commented 2 years ago

The changes mentioned have been committed.