-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
Comments
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. |
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. |
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. |
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. |
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. |
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
Load External Subtitles
0.3.1 loads .srt files by default but doesn't have the Load External Subtitles option so couldn't test that.
The text was updated successfully, but these errors were encountered: