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 basic GUI #11

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

Added basic GUI #11

wants to merge 5 commits into from

Conversation

j-waters
Copy link

@j-waters j-waters commented Apr 3, 2016

Additions

  • Added a basic GUI using easygui, that acts as a wizard to launch the spitter program.
  • GUI has the option to reformat a tracklist.
  • Changed the regex in the splitter program slightly, for my own needs.

I couldn't test it as much as I would have liked, using a file works fine, I didn't try with a YouTube link.

Lightning3105 added 3 commits April 3, 2016 17:51
Added a simple user interface using the 'easygui' module.
@crisbal
Copy link
Owner

crisbal commented Apr 3, 2016

Hey Lightning3105,

Well done!

I had to install an additional pip package to make the GUI work (easygui) which was not present in the requirements.txt file. Can you please add it (if you setup everything with virtualenv on Linux you should be able to do pip freeze > requirements.txt) ?

Lightning3105 added 2 commits April 3, 2016 22:28
@JohnMoutafis
Copy link
Contributor

Hey @Lightning3105 can you please resolve the conflicts in order for @crisbal to be able to merge your PR?

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.

None yet

3 participants