Skip to content

Add Function to Change/ADD subtitle and change the audio track #91

Merged
solid-software merged 9 commits into
solid-software:masterfrom
rikaweb:master
Aug 26, 2020
Merged

Add Function to Change/ADD subtitle and change the audio track #91
solid-software merged 9 commits into
solid-software:masterfrom
rikaweb:master

Conversation

@rikaweb

@rikaweb rikaweb commented Aug 5, 2020

Copy link
Copy Markdown
Contributor

Add Function to Change/ADD subtitle and change the audio track

@alr2413

alr2413 commented Aug 6, 2020

Copy link
Copy Markdown
Contributor

Hi,

Is it possible to return the name or text of audios & subtitles list in addition to the index?

Thanks.

@solid-software

Copy link
Copy Markdown
Collaborator

Great job, thanks for the PR @rikaweb!

Do you plan on adding iOS part for it?

@rikaweb

rikaweb commented Aug 6, 2020

Copy link
Copy Markdown
Contributor Author

Hi
@solid-software I just update it for android and test it on android, I am java developer and have no preference on ios. maybe we can publish it and ask the IOS developer to help.

@solid-software

Copy link
Copy Markdown
Collaborator

As we declare support for both mobile platforms (Android and iOS) I think it would be hard to merge without adding iOS part to it. Let's see if somebody can help here to add iOS part of it so we can merge.

@rikaweb

rikaweb commented Aug 6, 2020

Copy link
Copy Markdown
Contributor Author

Hi,

Is it possible to return the name or text of the audios & subtitles list in addition to the index?

Thanks.

HI @alr2413
The VLC use this function for audio and subtitle description :
/**
* Get the list of available SPU (subtitle and audio tracks).
*/
public TrackDescription[] getSpuTracks() {
return nativeGetSpuTracks();
}
So this method in 'FlutterVideoView.java' file can provide what you are searching for
https://stackoverflow.com/questions/48976386/libvlc-how-do-i-know-if-subtitles-text-track-are-external-or-part-of-video-co

Thanks

@mitchross

Copy link
Copy Markdown
Contributor

@solid-software I can do iOS part this week.... Is it possible to change this PR to a new feature branch on solid software repo? Forking a fork is really painful. If we make a new branch on solid software called, "feature/audio-track" , merge this PR into that branch, then I will fork the solid software repo, add in iOS. Finally we can PR the feature/audio branch to master.

@solid-software

Copy link
Copy Markdown
Collaborator

What if @rikaweb adds you to rikaweb:master so you can make the changes here? I think that might simplify it a bit.

@alr2413

alr2413 commented Aug 11, 2020

Copy link
Copy Markdown
Contributor

I think, it's better to have a dev-branch beside master branch for developing purpose.

@rikaweb

rikaweb commented Aug 11, 2020

Copy link
Copy Markdown
Contributor Author

What if @rikaweb adds you to rikaweb:master so you can make the changes here? I think that might simplify it a bit.

I can do that if needed!

@mitchross

Copy link
Copy Markdown
Contributor

What if @rikaweb adds you to rikaweb:master so you can make the changes here? I think that might simplify it a bit.

I can do that if needed!

Add me to your master, I'll make iOS changes

@rikaweb

rikaweb commented Aug 11, 2020

Copy link
Copy Markdown
Contributor Author

What if @rikaweb adds you to rikaweb:master so you can make the changes here? I think that might simplify it a bit.

I can do that if needed!

Add me to your master, I'll make iOS changes

Added!

@mitchross

Copy link
Copy Markdown
Contributor

@rikaweb I need a video file to test this with thats hosted online. How did you test your android changes?

@rikaweb

rikaweb commented Aug 20, 2020

Copy link
Copy Markdown
Contributor Author

@rikaweb I need a video file to test this with thats hosted online. How did you test your android changes?

I add subtitles to video first and then test the change subtitle option.

@mitchross

Copy link
Copy Markdown
Contributor

@rikaweb I need a video file to test this with thats hosted online. How did you test your android changes?

I add subtitles to video first and then test the change subtitle option.

Can you please provide specific instructions step by step? Local video? hosted video ? what code are you using to add subtitles? If you can provide me with this, I can do the ios code.

@rikaweb

rikaweb commented Aug 20, 2020

Copy link
Copy Markdown
Contributor Author

@rikaweb I need a video file to test this with thats hosted online. How did you test your android changes?

I add subtitles to video first and then test the change subtitle option.

Can you please provide specific instructions step by step? Local video? hosted video ? what code are you using to add subtitles? If you can provide me with this, I can do the ios code.

Please send me your email, I send the file to you in private

@mitchross

Copy link
Copy Markdown
Contributor

@rikaweb I need a video file to test this with thats hosted online. How did you test your android changes?

I add subtitles to video first and then test the change subtitle option.

Can you please provide specific instructions step by step? Local video? hosted video ? what code are you using to add subtitles? If you can provide me with this, I can do the ios code.

Please send me your email, I send the file to you in private

perfect.... mitchross09@gmail.com

@rikaweb

rikaweb commented Aug 20, 2020

Copy link
Copy Markdown
Contributor Author

@rikaweb I need a video file to test this with thats hosted online. How did you test your android changes?

I add subtitles to video first and then test the change subtitle option.

Can you please provide specific instructions step by step? Local video? hosted video ? what code are you using to add subtitles? If you can provide me with this, I can do the ios code.

Please send me your email, I send the file to you in private

perfect.... mitchross09@gmail.com

Sent!

@mitchross

Copy link
Copy Markdown
Contributor

@rikaweb iOS changes implemented

@dhruvkelawala

dhruvkelawala commented Aug 25, 2020

Copy link
Copy Markdown

I hope this changes are merged soon in the package and published. Thanks @rikaweb @mitchross

@solid-software solid-software merged commit 6fcee85 into solid-software:master Aug 26, 2020
@solid-software

Copy link
Copy Markdown
Collaborator

Great job - thanks @rikaweb @mitchross !

@dhruvkelawala

Copy link
Copy Markdown

How add a subtitle file (.srt) from network? @rikaweb @mitchross @solid-software

@rikaweb

rikaweb commented Sep 4, 2020

Copy link
Copy Markdown
Contributor Author

How add a subtitle file (.srt) from network? @rikaweb @mitchross @solid-software

you need first download the file then add it to VLC. VLC doesn't support network subtitles. you can use cache to that too.

@dhruvkelawala

Copy link
Copy Markdown

How add a subtitle file (.srt) from network? @rikaweb @mitchross @solid-software

you need first download the file then add it to VLC. VLC doesn't support network subtitles. you can use cache to that too.

Can't we make a helper function? Something like

controller.addSubtitle(Subtitle.fromNetwork(url));

Same with Subtitle.fromLocal(path)

@mitchross

Copy link
Copy Markdown
Contributor

How add a subtitle file (.srt) from network? @rikaweb @mitchross @solid-software

you need first download the file then add it to VLC. VLC doesn't support network subtitles. you can use cache to that too.

Can't we make a helper function? Something like

controller.addSubtitle(Subtitle.fromNetwork(url));

Same with Subtitle.fromLocal(path)

submit a PR, I will review

@alr2413

alr2413 commented Sep 9, 2020

Copy link
Copy Markdown
Contributor

I will submit a new PR and reference these issues there.

@alr2413

alr2413 commented Sep 11, 2020

Copy link
Copy Markdown
Contributor

How add a subtitle file (.srt) from network? @rikaweb @mitchross @solid-software

you need first download the file then add it to VLC. VLC doesn't support network subtitles. you can use cache to that too.

Can't we make a helper function? Something like

controller.addSubtitle(Subtitle.fromNetwork(url));

Same with Subtitle.fromLocal(path)

submit a PR, I will review

I will submit a new PR and reference these issues there.

@dhruvkelawala , @mitchross , @solid-software
Here is my PR #104.

@dhruvkelawala

Copy link
Copy Markdown

Yes... I saw it. Looks great. It's upto @solid-software and @mitchross to approve it.

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.

6 participants