Skip to content

Conversation

@cryeprecision
Copy link
Contributor

Hi,

I have this problem where I'd like to set sub-margin-x to values greater than 300 (the current hard limit). This happens in two cases:

  • My monitor is relatively big and even with a value of 300 at 16:9 the subtitles are (a little) too wide to comfortably read. A value of 300-400 worked best for me here.
  • When the video player window is very wide, e.g., because its in the lower half of a horizontal split in a tiling window manager, the sub-margin-x values required to get the subtitles to a comfortable width get very large. A value of 800-900 worked best for me here.

This PR increases the hard limit from 300 to 1200 since 1200 is about the largest I ever needed plus some headroom.

It might make sense to further increase this value or maybe introduce something like sub-max-size since people with some insanely wide monitors might need even larger values.

Also sorry if there's already some option that covers this...

@mrfragger
Copy link

not sure if this will help but for videos that are small and black bars occur this command setts the video width and subs will wrap to that limit rather than subs go over the black bars. Probably not applicable to this case though.

Meta+t cycle-values sub-margin-x '19' '50' '100' '125' '150' '175' '200' '250' '300' #! Subtitles > Format > Subtitle margin-x

d-= set autofit '100%x100%'; cycle-values geometry '50%:50%' '100%:100%' '0%:0%' ; cycle-values window-maximized 'no' 'yes' #! Editing > Edit Fancy Subs > set window to actual image size

@github-actions
Copy link

github-actions bot commented Sep 8, 2025

@cryeprecision cryeprecision changed the title sub: increase hard limit on margin-x sub: remove upper limits of margin-x and margin-y Sep 8, 2025
@kasper93 kasper93 merged commit 4a5a9a5 into mpv-player:master Sep 15, 2025
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants