Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subtitle size/styling does not match Crunchyroll. #66

Closed
ohitstom opened this issue Oct 31, 2022 · 13 comments
Closed

Subtitle size/styling does not match Crunchyroll. #66

ohitstom opened this issue Oct 31, 2022 · 13 comments
Labels
bug Something isn't working

Comments

@ohitstom
Copy link

Subtitles are too small, lack shadow, and have a thinner outline.

Screenshots
Screenshot_20221031-030943.png
Screenshot_20221031-031851.png

The second image is Crunchyroll.

Client (please complete the following information):

  • OS: Windows
  • Version Latest

Additional context
This needs fixing asap please, also an option to change the font would be awesome. I personally really like Roboto.

@bytedream
Copy link
Member

Which command are you using to download the episode?

@ohitstom
Copy link
Author

ohitstom commented Oct 31, 2022

Which command are you using to download the episode?

crunchy-v2.3.5_windows.exe archive https://www.crunchyroll.com/series/G6NQ5DWZ6/my-hero-academia -d "D:\Main Personal\Torrents\Anime\Batch\Boku no Hero Academia" -o "{title} S{season_number}E{episode_number}.mkv"

I had a Convo with someone on the discord server who had a similar issue, it's something to do with a flag in libass.

@bytedream
Copy link
Member

bytedream commented Oct 31, 2022

This is probably mostly up to your video player. If I play the videos in two different players (vlc & haruna) the subtitles are shown differently (they're also in german bc I can only download them in german :/)

vlc
image

haruna
image

crunchyroll.com
image

@ohitstom
Copy link
Author

ohitstom commented Oct 31, 2022

This is definitely not an issue with my video player I tried it on multiple players, this issue is only evident on mobile. Please check that. It's more the size than the styling, it's way too small.

@Askadelion
Copy link

i also use the german version and i use the MPV and on the APPLE TV PLEX it is a resize error some players compensate it by themselves but in aegisub ass there is a flag that fixes it called "ScaledBorderAndShadow" this is missing in the ass download. If it is set it looks the same in almost all players.

@bytedream
Copy link
Member

Do you have another example where this is the case?
As said I can't exactly reproduce bc I haven't access to your subtitle language on crunchy

@Askadelion
Copy link

for testing you can use aegisub set the subtitle size to 640x360 and open it in aegisub without resample then you will have the same effect

@ohitstom
Copy link
Author

Do you have another example where this is the case?
As said I can't exactly reproduce bc I haven't access to your subtitle language on crunchy

I've only ever experienced it using this cli, it's the same across all videos I've downloaded with it.

Subplease which also rip from Crunchyroll don't have this issue and play at the correct font size and border width on mobile on multiple media sources.

Do you use libass at all in the cli?
ScaledBorderAndShadow is what fixed it for the guy on discord.

Also please try the same sources on mobile with vlc OR mpv they both have the same output of a tiny font with little to no border.

@Askadelion
Copy link

grafik
grafik
grafik

@bytedream
Copy link
Member

Do you use libass at all in the cli?

Nope, first time that I've heard from such issues.

@Askadelion
Copy link

Nope, first time that I've heard from such issues.

most ignore it

@ohitstom
Copy link
Author

Would be awesome if you'd please look into it, even as an optional flag, it makes the subs basically unwatchable on mobile or some tvs!

@bytedream bytedream added the bug Something isn't working label Oct 31, 2022
bytedream added a commit that referenced this issue Oct 31, 2022
@bytedream
Copy link
Member

ScaledBorderAndShadows: yes for archive was added in v2.3.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants