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

Zimit videos that work fine in desktop browser fail to work on Android #512

Closed
Jaifroid opened this issue Dec 19, 2023 · 4 comments · Fixed by #561
Closed

Zimit videos that work fine in desktop browser fail to work on Android #512

Jaifroid opened this issue Dec 19, 2023 · 4 comments · Fixed by #561
Labels
bug/non-critical upstream Should be worked on in kiwix-js zimit

Comments

@Jaifroid
Copy link
Member

There is no difference in the underlying code, and this isn't the libzim issue with video on Android, because we're not using libzim for this extraction. So it's a little strange. I suspect a resolution or a codec issue, if the dynamic player is requesting a specific version for Android.

@Jaifroid
Copy link
Member Author

These videos also fail with the legacy reading mode on Android only, which strongly suggests that Android needs a different format of video, and that as this hasn't been scraped, it cannot be displayed. Unfortunately, I can't currently test this on the Android app, due to inability to install the APK despite my phone being in developer mode.

@Jaifroid
Copy link
Member Author

Now I've managed to install the APK, and I confirm that the videos in www.ready.gov don't play on the Android app either.

@Jaifroid
Copy link
Member Author

Jaifroid commented Feb 6, 2024

This is NOT fixed by #561. It needs to be fixed upstream in openzim/warc2zim#144.

@Jaifroid
Copy link
Member Author

Jaifroid commented Oct 4, 2024

This was fixed as reported in openzim/zimit#291.

@Jaifroid Jaifroid closed this as completed Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/non-critical upstream Should be worked on in kiwix-js zimit
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant