[solved] FFmpeg, Use Font Settings, and .ass Subtitles

[Madoka]

I had mention before that using FFmpeg for subtitles, checking the "Use Font Settings..." feature, and using .ass subtitles will result in tiny subtitles. I have some new information on this. This is on 5.5.0.

Most .ass subtitles will have the video resolution in the subtitle file in the form of:

Code: Select all

PlayResX: 848
PlayResY: 480

then the font size displayed for

Code: Select all

Style: Default,Arial Narrow,30,&H00AAFFFF,&H000000FF,&H00000000,&H80000000,-1,0,0,0,100,100,0,0,1,2,2,2,50,50,20,1

will be much smaller than it should be. However, if the .ass file does not specify the video resolution, then the font size will be correctly calculated and displayed. So it seems as if there is an issue when the video size is specified by the file itself when FFmepg calculates the font size.

Also, I noticed that checking "Use Font Settings..," will also result in the color set in the subtitles tab of the Common Transcode Settings page (subtitles_color =) being used for the "Default" style of subs. I think this should not be the case as the .ass file already specifies a color for those subs.

In the above example, if the color setting is left at "white" (FFFFFF), default for UMS, it will change the Default styled subtitles to white when they should be light yellow.

It does use that color for .srt subtitles, which is how I think I should work as then you can change the color from white to something else as .srt files generally do not specify color.

[valib]

Thanks for the comment. Fix was made in https://github.com/UniversalMediaServer ... 588f038f1b Can you try this latest version?

[Madoka]

Thanks for looking into this, valib!

With the new commit, with FFmpeg and "Use Font Settings.." checked,
1. Font size for .ass subtitles works correctly, even with the video resolution defined in the file.
2. The color set in UMS is no longer applied to the Default style font/subtitle.
3. Color still applied to .srt subtitles.

Thank you!!

[valib]

Thanks for the testing.

[Madoka]

Just a suggestion. If this change has been approved, then perhaps the tooltip should also be changed to remove the second line about settings being applied to .ass subs or changed to read that this setting does not affect .ass subs.

[valib]

Just a suggestion. If this change has been approved, then perhaps the tooltip should also be changed to remove the second line about settings being applied to .ass subs or changed to read that this setting does not affect .ass subs.

Thanks for the suggestion. I made it and it will be included in the next release.