beforeunload handler causes app to close on refresh from DevTools

I tracked it down to this: bc7b401

window.addEventListener('beforeunload', (e) => {
    // See http://electron.atom.io/docs/api/browser-window/#event-close
    e.returnValue = false;
    this.app.close();
});

We should find a better way to do this.

[martpie]

yep, it would call BrowserWindow.hide() and not minimize() though.

If the option is not checked in the admin, it will close the app.

You’ll need to edit some file like

• js/actions/AppActions.js
• js/main-process/tray.js

[YurySolovyov]

I think this is broken again/still. Dev flow like fix->refresh->check does not work for me. It either errors or minimizes the app.