bmathews / playback

experimental video player build using atom-shell
MIT License
0 stars 0 forks source link

Refactor TODO #1

Open bmathews opened 8 years ago

bmathews commented 8 years ago

MVP

ideal

bonus

bmathews commented 8 years ago

@mafintosh :wave:

Here's some of the work I've been doing to clean up the UI and split out the server/playlist into electron's main process, and add support for webchimera. There are lots of small improvements that improve stability for things like switching between chromecasts and local players, playlist behaviors, and much more.

Let me know what your thoughts are regarding this and then I can get it ready for a PR.

Code is on this branch: https://github.com/bmathews/playback/tree/refactor-chimera-rebase

image

image

image

image

mafintosh commented 8 years ago

WHOOOOOOOAH THAT LOOKS AWESOME

mafintosh commented 8 years ago

not sure how much it'll offend @maxogden if we remove the intro cat though

Kamahl19 commented 8 years ago

Great job @bmathews ! Few days ago I have experimented with AirPlay and I have made mafintosh's version working with XBMC AirPlay. I want to make it work with AppleTV too. I will gladly contribute with these features to this rewrite, once it is a bit stable or when I get it running. I was not able to run chimera-rebase on Win x64 (appearently some problem with 32 bit version of VLC libs on 64bit system).

bmathews commented 8 years ago

@Kamahl19, awesome! I'm hoping it's easy to add airplay support to this with the way the "players" have been split up.

Maybe you already looked and tried, but wcjs-prebuilt should work on Windows,. You might have to expiriment with the env variables during install? https://github.com/Ivshti/wcjs-prebuilt

Kamahl19 commented 8 years ago

@bmathews It will be much easier to add it to yours refactored app, the old one is quite a mess :D But still a nice piece of software.

Yea, I did this but it did not help SET WCJS_PLATFORM=win && SET WCJS_ARCH=x64 && SET WCJS_RUNTIME=electron && npm i wcjs-prebuilt

I suppose you are on Mac

bmathews commented 8 years ago

@Kamahl19 Hm, bummer. I am on a mac, yeah.

To get it to run in the meantime, you should be able to comment out the import WebChimera in app/front/UI.jsx and where it's instantiated in the UI's constructor at line 63. Then at line 54 of app.js, set it to use controller.PLAYER_HTML as the default instead.

I'll see if I can get a windows machine set up in the near future to try to troubleshoot that more.

max-mapper commented 8 years ago

im ok as long as the cat is incorporated in the drag/drop (as it is in the mock up)

max-mapper commented 8 years ago

oh also, that cat gif is unlicensed, we should probably switch to one of my cat photos (public domain) https://www.flickr.com/photos/catmapper/albums/72157662462294366

mafintosh commented 8 years ago

i bet @watson could make airplay work

bmathews commented 8 years ago

@mafintosh Just trying to get webchimera to work when packaging as a mac app, then I'll work on trying to get webchimera on windows, then I'll submit a PR.

RSATom commented 8 years ago

@bmathews, I've managed to setup prebuilt for WebChimera.js with embedded VLC binaries. Maybe it could be useful to you since wcjs-prebuilt is too unstable atm. https://github.com/RSATom/WebChimera.js/releases