This reverts commit 4702f695.

Duncan McNamara authored 2 months ago and Nicolas Pomepuy committed 2 months ago

On a new install, the manageMediaPermsCheck onclicklistener will
ask for video/audio permission for API 33+, but for 30+ goes to
getStoragePermission which ends calling requestStorageAccess.
When API 30+ this will spawn a new PermissionListDialog, creating a
loop.

In the onClickListener, asking for READ_EXTERNAL_STORAGE straight away
for API 30+ solves this.

Fixes #3137

Fixes #3132

Duncan McNamara authored 2 months ago and Nicolas Pomepuy committed 2 months ago

Revert of c49382e3. After some discussions, and user
complaints it was decided to go back to the old behavior and keep repeat
one enable when pressing next.

Most players do that. VLC for iOS also changed to this behavior

Fixes #3131

Nicolas Pomepuy authored 2 months ago and Duncan McNamara committed 2 months ago

Fixes #3095

Because fast seek is not precise, it will usually seek before the
requested time when seeking with abrepeat, causing the abrepeat to seek
again, and stucks the player in a loop.
Adding a small margin of time in which the seek is considered acceptable
when fast seek is enabled stops it from looping.

Duncan McNamara authored 2 months ago and Nicolas Pomepuy committed 2 months ago

Apparently I still don't know that the project is called VLC for Android
and not vlc-android ...

Duncan McNamara authored 2 months ago and Nicolas Pomepuy committed 2 months ago

Because of all the different permission APIs, it is easier to create a
File where we want to extract the subtitle file, and check if we have
write permission.

Fixes #3133

Nicolas Pomepuy authored 2 months ago and Duncan McNamara committed 2 months ago

Fixes #3130

Nicolas Pomepuy authored 2 months ago and Duncan McNamara committed 2 months ago

Fixes #3011

Nicolas Pomepuy authored 3 months ago and Duncan McNamara committed 2 months ago

Fixes #3124

Nicolas Pomepuy authored 2 months ago and Duncan McNamara committed 2 months ago

Fixes #3121

Duncan McNamara authored 2 months ago and Nicolas Pomepuy committed 2 months ago

This reverts commit 8880ccca.

Reverts the MR !1720
This MR was not finished, and made unecessary additions, causing
playqueues of one file to play twice sometimes. I should not have merged
it ...

Fixes #3116

Fixes #3128

Nicolas Pomepuy authored 2 months ago and Duncan McNamara committed 2 months ago

Fixes #3109