Black border around volume change bar and progress bar became much too thin

[mlindner]

mpv Information

mpv v0.38.0-582-ge8e6e91c49-dirty Copyright © 2000-2024 mpv/MPlayer/mplayer2 projects
 built on Jul  8 2024 14:19:51
libplacebo version: v7.349.0
FFmpeg version: n7.0.1
FFmpeg library versions:
   libavcodec      61.3.100
   libavdevice     61.1.100
   libavfilter     10.1.100
   libavformat     61.1.100
   libavutil       59.8.100
   libswresample   5.1.100
   libswscale      8.1.100

Other Information

• Linux version: Arch Linux
• Kernel Version: Linux archHTPC 6.9.8-arch1-1 The big pull request of pending changes #1 SMP PREEMPT_DYNAMIC Fri, 05 Jul 2024 22:11:24 +0000 x86_64 GNU/Linux
• GPU Model: 1f:00.0 VGA compatible controller [0300]: NVIDIA Corporation GA106 [Geforce RTX 3050] [10de:2507] (rev a1)
• Mesa/GPU Driver Version:
• Window Manager and Version:
• Source mpv: self-built
• Introduced in version: Unknown, what I'm looking to find

Reproduction Steps

Use on a 4k window. Use --no-config and change the volume level or seek. Observe that the border around the drawn OSD is much thinner than it used to be.

Expected Behavior

The bar should look something like:

Actual Behavior

Instead it looks like:

Log File

Not sure what log file would demonstrate this.

Sample Files

No response

I carefully read all instruction and confirm that I did the following:

• I tested with the latest mpv version to validate that the issue is not already fixed.
• I provided all required information including system and mpv version.
• I produced the log file with the exact same set of files, parameters, and conditions used in "Reproduction Steps", with the addition of --log-file=output.txt.
• I produced the log file while the behaviors described in "Actual Behavior" were actively observed.
• I attached the full, untruncated log file.
• I attached the backtrace in the case of a crash.

[guidocella]

--osd-bar-border-size

[mlindner]

--osd-bar-border-size

Yes I realize there's an option to change it. I'm asking why it changed. This is a regression.

[guidocella]

It was changed because it looks better.

[mlindner]

It was changed because it looks better.

Can you point to the commit/line that was changed? I want to know the manner of the change so I can precisely revert it.

And no it doesn't look better. The bar border size is practically invisible now because it is so thin. The border exists for a reason, so that the OSD doesn't blend into the background on white images. Try viewing it from a good distance away on a 4k TV.

[guidocella]

#12959
#12994

It looks better because the previous default was hideously thick. Most videos aren't white and even then you can still see the border just fine. Either way the option exists so you can customize it.

[mlindner]

#12959 #12994

It looks better because the previous default was hideously thick. Most videos aren't white and even then you can still see the border just fine. Either way the option exists so you can customize it.

What was the purpose of the float dent = MPMAX(border * 1.3, 1.6); change? Or rather, in detail, what is that doing?

Also, thank you, I set the value to 1 which seems to be a good match for the OSD and stats border thickness.

Also, I really suggest watching on a 4k tv, sat a good distance away from you, especially if you're somewhat nearsighted. It doesn't look good at all. It just looks like it's almost gone.

[guidocella]

dent is the size of chapter markers and is proportional to the border. Giving them a minimum value makes them clearly visible even with a smaller border.

[hooke007]

It shouldn't be an issue here.
Post the discussion instead.