Bug 464665

Summary: Question: can mpv CLI options be used with Haruna to add subtitle background?
Product: [Applications] Haruna Reporter: cipricus <cipricus>
Component: genericAssignee: george fb <georgefb899>
Status: RESOLVED NOT A BUG    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description cipricus 2023-01-22 20:18:33 UTC
Can mpv options (like this: https://mpv.io/manual/master/#options-sub-back-color) be used with Haruna?
Comment 1 george fb 2023-01-23 06:33:17 UTC
That's what the custom commands settings do. There's placeholder showing the format to use and a help page going into details.
Comment 2 cipricus 2023-01-23 07:59:36 UTC
(In reply to george fb from comment #1)
> That's what the custom commands settings do. There's placeholder showing the
> format to use and a help page going into details.

Thank you for the swift reply!

I have seen the format example, but it looked obscure to me, because it is different from what I had in mind (see below). The help is very limited.  The place holder has an example: `add volume +10`, which corresponds to the note in the help saying:

>Not all commands will work as some are specific for mpv.
>Most useful are the commands to manipulate properties, like set, add, cycle.

  In fact, I am trying to add subtitles background (https://mpv.io/manual/stable/#options-sub-back-color). I have tested that in mpv (`mpv --sub-back-color=#131415`) and it works as such, but not in Haruna. I have tried  ` --sub-back-color=#131415` also, doesn't work.

1. Is this type of option supposed to work?
2. If yes, what is the form of the command to be added according to the example above?

Thank you.
Comment 3 cipricus 2023-01-23 08:03:31 UTC
Please note that I don't want to have mpv as such always showing subs background, I would like to have that just in Haruna (and that triggered with a shortcut, if possible). Maybe I should post that as a different issue?
Comment 4 cipricus 2023-01-23 08:14:11 UTC
  ` sub-back-color=#131415`, doesn't work.
Comment 5 george fb 2023-01-23 08:15:52 UTC
(In reply to cipricus from comment #4)
>   ` sub-back-color=#131415`, doesn't work.

Why do you insist on the =, there is not = in the example.

Why is it so hard to understand?
"Most useful are the commands to manipulate properties, like set, add, cycle"
So `set`, `add` and `cycle` are commands.
In the example `add volume +10`, `add` is the command `volume` is the property and `+10` is the value.

Now all you have to do is replace them according to what you want to do, in your case `set sub-back-color #131415`.
Comment 6 cipricus 2023-01-23 09:07:41 UTC
Thank you! Sorry for my ignorance, I was thinking "set" to be a mpv option other than what can be associated with subtitles backgrounds, I was expecting something identical to arguments for mpv as such (like cellulid and smplayer do it).