{stats,osc}.lua: respect --osd-scale-by-window by default

[na-na-hi]

This lets these scripts scale the elements with OSD by default. The vidscale option now accepts and defaults to auto value which enables this behavior.

[guidocella]

I implemented this too but was waiting for the console one to get merged. Personally I would just remove vidscale unless someone can think of a use case of having each script scale differently, especially since the name of this option is wrong, they scale with the window and not with the video, which can be between black bars.

[github-actions]

Download the artifacts for this pull request:

Windows

• mpv-i686-w64-mingw32
• mpv-x86_64-w64-mingw32
• mpv-x86_64-windows-msvc

macOS

• mpv-macos-14-arm
• mpv-macos-12-intel
• mpv-macos-13-intel

[na-na-hi]

Personally I would just remove vidscale unless someone can think of a use case of having each script scale differently

This is kept for compatibility reason. If someone has already set vidscale to no but hasn't set --osd-scale-by-window to no, it no longer works.

Also since stats.lua contains too much information, there is a valid reason to set vidscale to no so that more content is shown if the window size is large without also changing --osd-scale-by-window.

[guidocella]

FWIW the OSC behavior will change by default anyway if we're going to default to --osd-scale-by-window=no (which has been approved at least by Dudemanguy).

[na-na-hi]

Changing the default of --osd-scale-by-window is going to be done later and it's out of scope for this PR, which doesn't change the behavior for --no-config. It merely provides a mechanism to respect that option, and a way to override that if the user wants to do so.

I don't see any value in removing this option from users if the mechanism isn't broken or doesn't become a significant maintenance burden.

[guidocella]

You could also add mp.observe_property("osd-scale-by-window", "native", update_scale) in stats.lua and mp.observe_property("osd-scale-by-window", "native", request_init_resize) in osc.lua (I didn't add this to console.lua because it updates anyway if run cycle osd-scale-by-window).

[na-na-hi]

Added osd-scale-by-window runtime change handling.

[kasper93]

I implemented this too but was waiting for the console one to get merged.

Since @Dudemanguy had strong opinion about this PR, I will let you guys decide what is the path forward. I personally think that scaling make sense, if you resize window often, so that OSC doesn't cover half of the video with small sizes. But as long as there is an option to control that, it will be ok.

[Dudemanguy]

Hmm no, I'm fine with this. It doesn't change current behavior.

[kasper93]

Hmm no, I'm fine with this. It doesn't change current behavior.

I refereed to #14114 which guido mentioned. This all seems like contented changes.

[Dudemanguy]

My general feeling on the matter is that scaling the scripts that are primarily text (e.g. stats and console) is basically useless at best. If you make the window small, then you can't read the text anyway which defeats the whole point. It makes more sense to just pick a sensible default font size imo and stick to it like what console currently does. For the osc, I am not so sure. There may be utility in scaling it up although scaling it too far down also makes it kind of useless.

Anyways, this PR doesn't change the status quo so no objections here.

[kasper93]

My general feeling on the matter is that scaling the scripts that are primarily text (e.g. stats and console) is basically useless at best. If you make the window small, then you can't read the text anyway which defeats the whole point.

I hear you, but there is a range of readability. Also if user makes player window small (and video) it means they can see it, presumably looking at the player up close. If the user make it big the situation is reversed. I mean scaling the text (and video) makes it usable for both close watching on small screen and far watching on big screen. Without scaling, both those cases would need different font size, manually managed by user. I think it makes little sense in practice to force it and I don't think users are complaining about it and imho it would be jarring change.

[kasper93]

LGTM. Please rebase.

[guidocella]

For the record Kasper and I agreed to keep the current defaults for the reasons he stated and for consistency with VLC and MPC-HC (I thought only mpv scaled text, but those do too, though only subtitles for MPC-HC, and only mpv scales the UI).