Add a compact mode to the OSC

[houssemko]

Expected behavior of the wanted feature

By moving the remaining time and total time in front and in the end of the progress bar , the OSC can be more compact

[Samillion]

What do you guys think about this layout?

Also solves #264

Changes:

• Reduce OSC footprint by re-organizing elements
• Move chapter_title below title
• Move time_stamp and cache_info

[Samillion]

Test script: compact_osc/modernz.lua
Commit: 058fb6e

Please test and let me know if there are any issues or changes that need to be made.

Changes:

• Reduced bottomhover_zone to 130
• Reduced time_font_size to 16
• Reduced raise_subtitle_amount to 125
• Reduced playpause_size to 28
• Reduced fade_alpha to 130
• Reduced possible OSC height to 140 and 100 (dependent on title function)
• Moved chapter_title below title
• Moved time stamps next to left side buttons
• Moved cache_info next to right side buttons
• Improved elements visibility based on window width

Pinging @Keith94 and @Xurdejl as they have shown interest in compact mode.

[da3dsoul1]

Wow, compact mode is so cool! Thanks!

[Samillion]

Updated: compact_osc/modernz.lua
Commit: 2ee659c

Changes:

• Increase chapter_title font size and padding
• Position title based on the existence of chapters to eliminate empty space with no chapters
• Move side and middle buttons to be more symmetrically centered
• Decrease volume bar width to 55

With chapter_title:

Without chapter_title:

If above change is made, the time can be moved to the left which creates more room.

Wouldn't this break "standard"? Most OSCs (local and online players) have [vol_icon] [vol_bar] [time] or move the volume bar all the way to the right.

I'm not rejecting this specific suggestion, I'm just concerned on user experience.

Do you think we should instead make the volume bar a user option? ie: show volume icon only if need be.

[Samillion]

Updated script. (Same link)
Commit: 52b77e6

Changes:

• Increase chapter_title padding
• Add ↳ as a chapter_title prefix

I tested them, that prefix is the only one that is kind of visible lol. Not sure if I should keep it or not, to be honest.

[Samillion]

Updated: compact_osc/modernz.lua
Commit: c75e35e

Changes:

• Fix chapter_title 1 pixel offset
• Improve chapter check for title position by checking user_opts.show_chapter_title as well
• Remove chapter_title prefix

[Samillion]

Updated: compact_osc/modernz.lua
Commit: 8331357

Changes:

• Combined time codes into one element for consistency time_codes
• Changed click functions on time code
  • Left click: show milliseconds (ms)
  • Right click: show remaining time
• Display cache_info and cache_info_speed in two separate lines
• Display cache_info correctly aligned if cache_info_speed is disabled
• Remove Cache: label and use tool tip on hover instead locale.cache
• Display current volume on hover with vol_ctrl icon

[Samillion]

For local videos the cache speed isn't centered vertically. (Is such info useful for local videos?...hmm)

I asked the same question on the feature request for cache, and they said to enable for all types. I also only use cache speed for online videos only, so I enable it with my Stream auto profile

The height and width hit box for the new timecodes seem a tiny bit off. Needs more height and a little less width.

Will try to fix as best as I can.

The distance between cache info and the right buttons keeps changing as cached time and speed change. Since it's on the right side, would it be better to align to the right,

Great idea. I'll use an=6 (align right), looks much better that way.

Another suggestion: since it's using tc_left / tc_right format for time codes, can we make the tc_right always display total time and toggle tc_left for time played and time remaining?

The mpv users would crucify me if I did that lol. We can maybe introduce a feature in the future to customize time display behavior

[Samillion]

Updated: compact_osc/modernz.lua
Commit: 138e263

Changes:

• Decrease total height of OSC to 130
• Dynamically decrease height of OSC based on title or chapter_title existence/function with osc_height_offset
• Improve time_codes hitbox height and width
• Improve cache_info and cache_info_speed position and alignment
• Reverse time_codes click functions

[Keith94]

Everything looks great. Except I'm a little confused. When cache info is enabled why does the cache time show up invisible on local videos?

[Samillion]

So cached time is not showing at all for you on local videos?

[Keith94]

So cached time is not showing at all for you on local videos?

Yeah, not at all.. It used to show just the speed properly and cache time was invisible. Commit 87478e5 made it so speed doesn't show up now. Feels like I made a simple mistake somewhere lol.

[Samillion]

Yeah, not at all.. It used to show just the speed properly and cache time was invisible. Commit 87478e5 made it so speed doesn't show up now. Feels like I made a simple mistake somewhere lol.

Hmmm. Could it be related to your cache options in mpv.conf? Mine is:

### Cache
cache=yes
demuxer-max-bytes=400MiB
demuxer-max-back-bytes=200MiB

Also, in my media auto profile it changes to:

[Media]
    profile-desc=Apply options for Movies and TV Shows directories
    profile-cond=get('path', ''):match('\\Movies\\') or get('path', ''):match('\\TV Shows\\') or get('path', ''):match('^X:\\')
    profile-restore=copy-equal
    demuxer-max-bytes=1500MiB
    demuxer-max-back-bytes=500MiB

[Keith94]

Ahh, thank you. You found my simple mistake.

I left mpv's cache option alone which defaults to auto. When I set it to yes I can see the cache finally.

I guess if you set yours to "auto" or "no" then you should be able to reproduce that tooltip over invisible cache thingy. Hopefully.

[Samillion]

I guess if you set yours to "auto" or "no" then you should be able to reproduce that tooltip over invisible cache thingy. Hopefully.

Good point. Tooltip should only show if cache is shown.

[Samillion]

Updated: compact_osc/modernz.lua
Commit: 245127d

Changes:

• Display cache tooltip only if cache ranges exist.

@Keith94 I learned something new because of this bug. If you want to display cache_info for online only, then in mpv.conf set cache=auto

[Keith94]

That's great! 😎 I feel like modernz should follow that same setting, if possible. Don't show cache for local files unless you specifically have it on yes.

Edit; nvm, I think it may already be doing that. It's late.