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

electron wrapper #64

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

Conversation

fridgerator
Copy link

closes #63

Electron wrapper for reddit music player.

Most of the code was generated by electron-boilerplate. Very little was changed, other than removing unused scrips / styles / html.

  • electron/src/background.js line 38 just opens https://reddit.musicplayer.io directly
  • electron/src/background.js lines 56-70, are the keybindings which just fire click events on the prev, pause / play, next buttons.

Requires python 2.7 because of node-gyp. Other than that, dev environment with npm start. Build (unsigned) installers with npm run release.

@Illyism
Copy link
Member

Illyism commented Jan 8, 2017

Looks good. Gonna test it out on a Windows and Linux and see if it works...

@fridgerator
Copy link
Author

Right on. Let me know if there are any issues.

electron-boilerplate uses electron-builder, which supports creating installers on all 3 major platforms.

@Illyism
Copy link
Member

Illyism commented Jan 23, 2017

Works fairly well. I'm going to add a way to open external links in a separate browser. And a popup for the Reddit log in. Otherwise, this could use some extra desktop-only features that I'll look into.

@fridgerator
Copy link
Author

let me know if I can help with any features

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.

question: any interest in a standalone desktop app?
2 participants