-
-
Notifications
You must be signed in to change notification settings - Fork 457
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
add seperate button to adjust subtitle delay (if subtitles are present). #1731
add seperate button to adjust subtitle delay (if subtitles are present). #1731
Conversation
7f96d73
to
406973e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This option only works for LibVLC right now. It must also support ExoPlayer.
...c/main/java/org/jellyfin/androidtv/ui/playback/overlay/action/AdjustSubtitleDelayAction.java
Outdated
Show resolved
Hide resolved
...main/java/org/jellyfin/androidtv/ui/playback/overlay/CustomPlaybackTransportControlGlue.java
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/androidtv/ui/SubtitleDelayPopup.java
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/androidtv/ui/AudioDelayPopup.java
Outdated
Show resolved
Hide resolved
ExoPlayer does not support modifying subtitle timings (see here). The reason why I chose the same type of solution as with the current audiodelay feature. |
Ok makes sense to only add it for libvlc then |
406973e
to
284651e
Compare
Ok, so new files are converted to kotlin. I've struggled a bit with merging the two buttons into a single button. But I got it working. I've kept that as a seperate commit but I can squash them if desired. |
...main/java/org/jellyfin/androidtv/ui/playback/overlay/CustomPlaybackTransportControlGlue.java
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/androidtv/ui/playback/overlay/action/SettingAction.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/androidtv/ui/playback/overlay/action/SettingAction.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/androidtv/ui/playback/overlay/action/SettingAction.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/androidtv/ui/playback/overlay/action/SettingAction.kt
Outdated
Show resolved
Hide resolved
Fix magic numbers Fix formatting Remove superfluous member variables
Move all logic from `init` to constructor.
I've pushed some final changes so we can merge:
Thanks for your contribution! |
Changes
Issues
fixes #1636
fixes #1234
fixes #640
test
further improvement
It would probably be nicer to share functionality with the audio delay button: but I'm unsure what would be the nicest way to achieve that. My guess would be:
delay-popup.xml
instead of seperateaudio-delay-popup.xml
andsubtitle-delay-popup.xml
AdjustDelayAction
instead of two seperate onesAdjustDelayPopup
which determines if one (or both) settings should be shown & adjusts size accordingly.Not entirely sure if that would work. And my gut feeling says that there should be a way to keep the two
.xml
files seperate & add a level in between that merges the two. Not entirely sure how though. Input appreciated.