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

Doesn't load external .srt by default and can't locate any to load manually 0.3.2/0.4.2(non android 11 version) #3

Closed
Suakwe opened this issue Feb 10, 2021 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@Suakwe
Copy link

Suakwe commented Feb 10, 2021

Samsung Galaxy S7 running Android 8

When selecting a video through Play Local Media you can see both the video and .srt files, but when clicking Load External Subtitles no files can be located, there's no dropdown arrow to change location away from recent, and the "open from" menu doesn't show the phone's local storage.

Play Local Media

ss1
ss2

Load External Subtitles

ss3
ss4

0.3.1 loads .srt files by default but doesn't have the Load External Subtitles option so couldn't test that.

@arianneorpilla
Copy link
Owner

arianneorpilla commented Feb 11, 2021

Greetings, external .SRT files no longer loading by default is an artifact of changes I needed to make in order to get the app running on Android 11. Due to changes to scoped storage on Android 11, it is impossible for the external .SRT files to be known to the app without picking a separate file with the file picker once more after you select the video file. I did not find this intuitive at all, so I implemented the "Load External Subtitles" option.

Truth be told, I myself use the app on 0.3.1 on my own Android 10 device because I'm not satisfied with how files have to be accessed in newer versions of the app, and plan to separate the way files are handled below and on/above Android 11 (to use the older and newer system respectively, as I find the load by default feature convenient myself).

The way .SRT files aren't showing in your menu are a separate issue, however. I will look into it as soon as I can.

@Suakwe
Copy link
Author

Suakwe commented Feb 11, 2021

Ahh no worries, wasn't sure if that was the case or if only the Android 11 version wasn't able to load by default so figured I'd mention it.

Tried troubleshooting by moving some video and .srt files to an SD card to see if that could change anything but still couldn't locate any files. Did find a new issue however, one .mkv file I tried would play audio and show embeded subtitles, but the video was just black with horizontal line artifacting; the video also had this issue when testing it on mpv for Android, but I was able to fix it on that by turning off Hardware Decoding. I can open a new issue if you think its worth looking into.

@arianneorpilla
Copy link
Owner

arianneorpilla commented Feb 11, 2021

I can't test beyond what videos and devices I have, so thank you for the initiative. Other users seem to be having trouble with Matroska videos, some not playing at all and now what you've just discovered.

I'm motivated to switch from ExoPlayer which I am currently using right now to VLCKit which should allow videos to behave and be decoded exactly like they are on VLC -- it's working but there's just one issue that prevents me from releasing a build just yet (auto-rotate is not very smooth with it).

Thanks again for the heads up.

@arianneorpilla
Copy link
Owner

I have fixed the issue with .SRT files not showing via the "Load External Subtitles" option and in addition added support for .ASS files in 0.4.3.

Unfortunately, I'm unable to migrate the app to VLCKit as it's not working as elegantly as I want it to. I'll update when I have details but for now, I'm closing this as the relevant issue is fixed.

@arianneorpilla arianneorpilla added the bug Something isn't working label Mar 4, 2021
@arianneorpilla arianneorpilla reopened this Apr 4, 2021
@arianneorpilla
Copy link
Owner

I am briefly reopening this issue to announce that I have recently migrated the app to use VLCKit in the 0.6.0 release and that the videos you are having issues with should now be compatible.

If there are any further issues, please let me know. And with that, I am closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants