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

Resolve some android seeking issues #323

Merged
merged 1 commit into from
May 17, 2019
Merged

Resolve some android seeking issues #323

merged 1 commit into from
May 17, 2019

Conversation

bugfixin
Copy link
Contributor

@bugfixin bugfixin commented May 16, 2019

I believe this allows android to seek, but I think there's more to it than this quick fix - the cordova app seems to be getting transcodes / remuxes when it is not fully necessary which makes this bug worse.

Changes
Prevents a removal of the startPositionTicks variable so that the progress bar is not reset to 0:00 when seeking

Issues
#263
possibly jellyfin/jellyfin-android#133

Copy link
Member

@dkanada dkanada left a comment

Choose a reason for hiding this comment

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

This solved the issue on my phone, and I also tested seeking on several browsers so I don't believe it breaks anything. If no one else reports any issues I'd say it's okay to merge.

@joshuaboniface joshuaboniface merged commit b0f7a9b into jellyfin:release-10.3.z May 17, 2019
@fromage9747
Copy link

Where can I get the apk for this tracking issue fix? I am using the 0.9.0 release and still experience the inability to skip or resume a video.

@anthonylavado
Copy link
Member

@fromage9747 The last upload of 0.9.0 used an old build of certain modules for some reason. We will be making a new one soon.

@ckcr4lyf
Copy link
Contributor

ckcr4lyf commented Jun 8, 2019

So essentially, the new apk build will have the seeking issue fixed? Because for me as well, this is the biggest drawback right now.

@dag420
Copy link

dag420 commented Jun 9, 2019

plz seeking is important to me plz fix

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.

7 participants