hama3254 / Crunchyroll-Downloader-v3.0

Downloader for Crunchyroll
844 stars 108 forks source link

"ScaledBorderAndShadow:yes" in subtitles scripts #764

Closed MetalBat28 closed 1 year ago

MetalBat28 commented 1 year ago

Is your feature request related to a problem? Please describe. I've noticed CR subs don't have the ScaledBorderAndShadow: yes header under the [Script Info], which leads to an incorrect display of them, because borders and shadows are not scaled correctly.

Without ScaledBorderAndShadow: yes: Screenshot 2023-06-17 130226 mpv-shot0083

With ScaledBorderAndShadow: yes: Screenshot 2023-06-17 125128 mpv-shot0085

Describe the solution you'd like Could it be possible for this downloader to automatically add the ScaledBorderAndShadow: yes header in each subtitles script?

Describe alternatives you've considered It's not really an issue for me since I use mpv with some filters applied and so the subs are shown correctly. But using VLC, mpv without filters or other video players could lead to this problem. Adding the ScaledBorderAndShadow: yes or setting the correct resolution in PlayRes{X,Y} should fix the problem for all video players.

Additional context I want to clarify that it's not downloader's fault (it works fine) but of CR subtitles.

hama3254 commented 1 year ago

The downloader is not build to modify the subtitles. This leads to the issue that there are 3 ways the downloader access the subtilte file from CR.

Without merged subs i use curl to get the subtitles as text and save them, here i could simply modify the data.

Merged with ffmpeg - Default download gives the urls for the video and subtitles to ffmpeg so the downloader is not in between the output and the source.

Merged with Hybrid Mode or Hybrid Mode - keep cache lets the downloader copy the input before getting it to ffmpeg for processing, here i also could add the line.

I probaly add a setting (Default - Off) for this on Output settings tab where i can toggle it based on those factors, this way i only have to modify the data at the hybrid mode and unmerged part without having to deal with any ffmpeg comand directly.

hama3254 commented 1 year ago

it seems to work for me grafik

MetalBat28 commented 1 year ago

I tried with the Hybrid Mode and the new option activated and it worked! Thank you!