Skip to content
Snippets Groups Projects

Ability to play external subs and audio

Merged Rizky Maulana requested to merge Perklone/vlc-ios:external-sub into master

Currently, when watching a video through the video player, the app can only detect subtitles containing the video file's name. (e.g. Bao.mp4 and Bao.srt or Bao1.srt)

This feature allows the user to select any subtitles from the Files application in iOS and add them as a permanent subtitle for that video file. It is compatible with both local files and remote files through servers.

Current Issue: Adding subtitle for a remote file currently works when the application is still active, even when the video player is closed, but do not last more than one lifecycle (when the application is closed and then re-opened, the subtitle is gone from the tableView).

LocalFile1

FilePersistence

Close #391 (closed) and #1033 (closed)

Edited by Rizky Maulana

Merge request reports

Merge request pipeline #382142 passed

Merge request pipeline passed for 1d2b61c6

Merged by Felix Paul KühneFelix Paul Kühne 1 year ago (Sep 7, 2023 7:43am UTC)

Merge details

Pipeline #382145 passed

Pipeline passed for 1d2b61c6 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Felix Paul Kühne
  • Looks good so far! Remarks inline.

  • Rizky Maulana added 1 commit

    added 1 commit

    • 5fb06dad - VLCPlaybackService: Added Localized Text for Select Subtitles From Files

    Compare with previous version

  • Rizky Maulana added 1 commit

    added 1 commit

    • a336c233 - VideoPlayerViewController: Added feature for adding external subtitle for local and remote videos

    Compare with previous version

  • added Player label

  • Felix Paul Kühne changed milestone to %3.5.0

    changed milestone to %3.5.0

  • Felix Paul Kühne resolved all threads

    resolved all threads

  • Felix Paul Kühne
  • Rizky Maulana resolved all threads

    resolved all threads

  • Rizky Maulana added 1 commit

    added 1 commit

    • 150ee0c1 - VideoPlayerViewController: Added feature for adding external subtitle for local and remote videos

    Compare with previous version

  • Felix Paul Kühne
  • Rizky Maulana resolved all threads

    resolved all threads

  • Rizky Maulana added 2 commits

    added 2 commits

    • db6d503b - still not working
    • e156f158 - VideoPlayerViewController: Added if statement to check whether if a file is remote

    Compare with previous version

  • Rizky Maulana added 1 commit

    added 1 commit

    • d2d2ec88 - VideoPlayerViewController: Added feature for adding external subtitle for local and remote videos

    Compare with previous version

  • Rizky Maulana added 1 commit

    added 1 commit

    • 80bd8c16 - VideoPlayerViewController: add Audio Support

    Compare with previous version

  • Rizky Maulana changed title from Draft: Ability to play external subs to Draft: Ability to play external subs and audio

    changed title from Draft: Ability to play external subs to Draft: Ability to play external subs and audio

  • Rizky Maulana changed the description

    changed the description

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading