-
-
Notifications
You must be signed in to change notification settings - Fork 446
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
Strange placement of subtitles #5306
Comments
I don't use subtitles 90% of the time, so I only ran into this problem yesterday, but it may have been around for a while |
That's probably due to #5284, although I don't understand why this happens since even though the captions support more styling now, that shouldn't change their position ... |
After careful consideration, I think the best approach is to leave this as is. By changing the ViewType to VIEW_TYPE_WEB we shifted the control of captions to the captions provider. It is generally a good thing, since they can choose to show captions vertically, place them in a position that does not block important visual content, etc. To center the captions at all time forcefully (I'm not sure if it is possible - since we would need to tweak the CSS coming along with the captions, and I'm not sure we can access that), would be to no longer support the improved experience that lead to #5284. |
At least it doesn't push them to the left or right, covering half the screen. I'm perfectly fine with subtitles not being centred, but aligned to the left or right, while covering most of the bottom and not being almost vertical. TBH, I don't think many creators do properly formatted and aligned subtitles, 99% just provide text with basic punctuation, so I don't think it would be a big deal if some subtitles were displayed with HTML tags while others were displayed normally, because I feel that the way they are displayed now is more likely to irritate those who rely on them and cause more inconvenience than if formatted subtitles were not displayed perfectly. So, in general, I think it would be appropriate to at least disable "web" subtitles when the "System caption style" option is enabled. Not perfect, more of a temporary solution, but still better than what we have now. |
Fair point. I agree with your suggestion. Maybe adding a setting to enable/disable the feature is a good approach. If Bnryo thinks this is the way to go, I would happily address this ticket. |
That's okay with me, feel free to open a PR for adding the preference (something like "Rich caption rendering" or similar). I guess we should disable the preference by default, so that the default behavior is the one before the PR linked above.
Bnyro* 👀 |
Perfect, I'll grab the ticket then! 👍 |
Steps to reproduce
Expected behavior
They should display in the centre, or as specified in system settings:
Actual behavior
They are displayed in the right side of the screen:
LibreTube version
LibreTube Debug 0.20.1 commit 3a576a9
Android version
Android 13
Other details
No response
Acknowledgements
The text was updated successfully, but these errors were encountered: