diff options
| author | wm4 <wm4@nowhere> | 2015-01-29 20:28:39 +0100 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2015-01-29 20:28:39 +0100 |
| commit | 1cd11c43ee855eecc49fe43853450854ffc8b900 (patch) | |
| tree | 37ccfd69c291ec7b2a4c286c667f426f1b766ac5 | |
| parent | 547fd2a2299342a6ede3a09cf678b6f9932a0450 (diff) | |
| download | mpv-1cd11c43ee855eecc49fe43853450854ffc8b900.tar.bz2 mpv-1cd11c43ee855eecc49fe43853450854ffc8b900.tar.xz | |
command: change sub-visibility OSD message
Now it shows one of:
- "Subtitles hidden" (sub-visibility=no)
- "Subtitles visible" (sub-visibility=yes, sub!=no)
- "Subtitles visible (but no subtitles selected)" (otherwise)
It should be a bit more self-explanatory than before. On the other hand,
I have no clue about UI issues.
This also gets close to what's reasonably possible with the OSD
expansion string syntax, which is why it looks so awful.
| -rw-r--r-- | player/command.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/player/command.c b/player/command.c index 8779ea8c0a..ab7e973b2e 100644 --- a/player/command.c +++ b/player/command.c @@ -3636,7 +3636,8 @@ static const struct property_osd_display { { "secondary-sid", "Secondary subtitles" }, { "sub-pos", "Sub position" }, { "sub-delay", "Sub delay" }, - { "sub-visibility", "Subtitles" }, + { "sub-visibility", .msg = "Subtitles ${!sub-visibility==yes:hidden}" + "${?sub-visibility==yes:visible${?sub==no: (but no subtitles selected)}}" }, { "sub-forced-only", "Forced sub only" }, { "sub-scale", "Sub Scale"}, { "ass-vsfilter-aspect-compat", "Subtitle VSFilter aspect compat"}, |