Skip to content
This repository has been archived by the owner on Dec 30, 2021. It is now read-only.

Closing window on OSX kills radiant player #13

Open
madeleineostoja opened this issue Sep 7, 2016 · 2 comments
Open

Closing window on OSX kills radiant player #13

madeleineostoja opened this issue Sep 7, 2016 · 2 comments

Comments

@madeleineostoja
Copy link

madeleineostoja commented Sep 7, 2016

Closing the Radiant Player window (just closing, not quitting app) kills the player. When I try to open the app again it throws an exception:

Uncaught Exception:
TypeError: Cannot read property 'webContents' of null
    at send (/Applications/Radiant Player.app/Contents/Resources/app/backend.js:816:40)
    at Object.emit (/Applications/Radiant Player.app/Contents/Resources/app/backend.js:219:13)
    at EventEmitter.<anonymous> (/Applications/Radiant Player.app/Contents/Resources/app/backend.js:855:26)
    at emitTwo (events.js:106:13)
    at EventEmitter.emit (events.js:191:7)
    at /Applications/Radiant Player.app/Contents/Resources/app/node_modules/osx-mouse/index.js:11:9

On El Cap v10.11.6


As an aside, is this project still going ahead? Even in pre-alpha or whatever it's in it looks really promising (this breaking bug aside, obviously). Perf already miles better than old Radiant Player, and no Flash. Would happily use this as my daily driver if just the current limited featureset was stable.

@jacobwgillespie
Copy link
Member

Still planning to go ahead yep - I'm fairly busy with full-time employment and part-time school, but definitely planning to finish this up as the "2.0" version of Radiant.

Thank you so much for testing the early release and for the bug report - it's really helpful! I'll see what I can do to get it fixed. I'm assuming we want the close button to "hide" the player window but continue playing music, so I'll see what can be done there...

@MarshallOfSound
Copy link

@jacobwgillespie You can call preventDefault on the close event 👍

mainWindow.on('close', (e) => {
  e.preventDefault();
  mainWindow.hide();
});

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants