Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@svobs
Copy link
Contributor

@svobs svobs commented Jun 25, 2024


Description:

Adds explicit support for --shuffle=yes when supplied via the Additional mpv options table in Settings > Advanced.

As noted in this comment:

Fwiw, shuffle: yes WAS working for me, as of IINA for Mac 1.3.4, but it broke again in 1.3.5. :(

Analysis:

  1. PR Send directory arg to mpv instead of processing in IINA (fixes #4434) #4439 was the original attempt to add support for --mpv-shuffle arg via iina-cli. It was released in IINA 1.3.4. This also allowed --shuffle=yes via Settings > Advanced to work. But this introduced a regression: issue Opening folder spawns playlist that includes non-media files #4548
  2. PR Revert change to folder filtering + add special handling for shuffle arg instead (fixes regression: #4548) #4566 fixed the regression while adding support for --mpv-shuffle arg via iina-cli. This was in IINA 1.3.5. But it unfortunately broke support for --shuffle=yes via Settings > Advanced.
  3. This PR extends the previous PR to re-add support for --shuffle=yes using the improved solution.

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.

1 participant