Skip to content

Feat: Add subtitle delay feature#158

Merged
anilbeesetti merged 4 commits intomainfrom
subtitle-delay-feature
Jan 31, 2026
Merged

Feat: Add subtitle delay feature#158
anilbeesetti merged 4 commits intomainfrom
subtitle-delay-feature

Conversation

@anilbeesetti
Copy link
Owner

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a subtitle delay and speed adjustment feature to the media player by introducing a custom text renderer with offset capabilities. The implementation uses an abstract OffsetRenderer class to handle timing adjustments and integrates it with ExoPlayer through a custom NextTextRenderer.

Changes:

  • Introduced OffsetRenderer abstract class to provide time offset and speed adjustment capabilities for renderers
  • Implemented NextTextRenderer that extends OffsetRenderer and wraps the default TextRenderer with custom timing logic
  • Modified NextRenderersFactory to use the custom NextTextRenderer instead of the default text renderer

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.

File Description
media3ext/src/main/java/io/github/anilbeesetti/nextlib/media3ext/rendrer/OffsetRenderer.kt New abstract class providing timing offset/speed adjustment functionality with helper extension function
media3ext/src/main/java/io/github/anilbeesetti/nextlib/media3ext/rendrer/NextTextRenderer.kt Custom text renderer implementation with ExoPlayer extension properties for subtitle delay and speed control
media3ext/src/main/java/io/github/anilbeesetti/nextlib/media3ext/ffdecoder/NextRenderersFactory.kt Override of buildTextRenderers() to integrate the custom NextTextRenderer

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@anilbeesetti anilbeesetti merged commit f11ca49 into main Jan 31, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants