Skip to content
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

Added option to use short (two letter) language codes for the subtitle extension #41

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Sebazzz
Copy link
Contributor

@Sebazzz Sebazzz commented Dec 11, 2013

I added an option to use the short two-letter language code for the subtitle extension instead. For example, dutch subtitles would have the 'nl' extension instead of 'dut'.

Also, I refactored and commented the interface code, so it is easier to add new interface items.

Sebazzz and others added 4 commits December 11, 2013 20:12
…e extension, and also refactored and commented the interface code for easier extension
@adrianparisi
Copy link
Contributor

Hi Sebastian, that's a great functionality. I also would like to put this feature on the code and I tried by my self but I have a little bug yet when I apply the changes, the plugin is selecting a wrong option.

So if your code is ok it would be great to use it or if you can help me with mine may be we can finish it.

https://github.com/adrianparisi/vlsub

@Sebazzz
Copy link
Contributor Author

Sebazzz commented Apr 26, 2015

I will try to merge from upstream master, then commit again. Hold on :)

Sebazzz added 2 commits April 26, 2015 19:15
…e extension, and also refactored and commented the interface code for easier extension
@Sebazzz
Copy link
Contributor Author

Sebazzz commented Apr 26, 2015

Alright, I have rebased on top of master.

@adrianparisi
Copy link
Contributor

Ok thanks I'm going to take a look.

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.

4 participants