Readded the Option to hide the window on close for OSX

This commit is contained in:
Martin Gondermann 2016-07-03 16:12:04 +02:00
parent f35393d73e
commit 2b4ee5f0bc
5 changed files with 39 additions and 4 deletions

View file

@ -15,7 +15,7 @@
#### OS X
- Fixed that two icons appear on a notification.
- The main window is now hidden from dock on close
- Added Option to hide Window from dock on close
### Improvements
- Added shortcuts

View file

@ -124,6 +124,8 @@ The Settings Page is available from the **File** menu under **Settings** (Click
This option allows such images to be rendered, but please be careful for security.
- **Start app on login** (Windows, Linux)
- This option starts the application when you login.
- **Leave app running in notification area when the window is closed** (OS X)
- This option hides the window from the dock, if the window is closed
- **Toggle window visibility when clicking on the tray icon** (Windows)
- If checked, then a click on the system tray icon leads to a toggling of the minimized/maximized state of the window

View file

@ -61,6 +61,7 @@ var SettingsPage = React.createClass({
trayIconTheme: this.state.trayIconTheme,
disablewebsecurity: this.state.disablewebsecurity,
version: settings.version,
minimizeToTray: this.state.minimizeToTray,
toggleWindowOnTrayIconClick: this.state.toggleWindowOnTrayIconClick,
notifications: {
flashWindow: this.state.notifications.flashWindow
@ -101,9 +102,16 @@ var SettingsPage = React.createClass({
});
},
handleChangeShowTrayIcon: function() {
var shouldShowTrayIcon = this.refs.showTrayIcon.getChecked();
this.setState({
showTrayIcon: this.refs.showTrayIcon.getChecked()
showTrayIcon: shouldShowTrayIcon
});
if (process.platform === 'darwin' && !shouldShowTrayIcon) {
this.setState({
minimizeToTray: false
});
}
},
handleChangeTrayIconTheme: function() {
this.setState({
@ -115,6 +123,14 @@ var SettingsPage = React.createClass({
autostart: this.refs.autostart.getChecked()
});
},
handleChangeMinimizeToTray: function() {
var shouldMinimizeToTray = (process.platform !== 'darwin' || this.refs.showTrayIcon.getChecked())
&& this.refs.minimizeToTray.getChecked();
this.setState({
minimizeToTray: shouldMinimizeToTray
});
},
handleChangeToggleWindowOnTrayIconClick: function() {
this.setState({
toggleWindowOnTrayIconClick: this.refs.toggleWindowOnTrayIconClick.getChecked()
@ -164,6 +180,11 @@ var SettingsPage = React.createClass({
/>);
}
if (process.platform === 'darwin') {
options.push(<Input key="inputMinimizeToTray" id="inputMinimizeToTray" ref="minimizeToTray" type="checkbox" label="Leave app running in notification area when the window is closed"
disabled={ !this.state.showTrayIcon } checked={ this.state.minimizeToTray } onChange={ this.handleChangeMinimizeToTray } />);
}
if (process.platform === 'win32') {
options.push(<Input key="inputToggleWindowOnTrayIconClick" id="inputToggleWindowOnTrayIconClick" ref="toggleWindowOnTrayIconClick" type="checkbox" label="Toggle window visibility when clicking on the tray icon."
checked={ this.state.toggleWindowOnTrayIconClick } onChange={ this.handleChangeToggleWindowOnTrayIconClick } />);

View file

@ -391,8 +391,10 @@ app.on('ready', function() {
break;
case 'darwin':
mainWindow.hide();
if (config.minimizeToTray) {
app.dock.hide();
mainWindow.isHidden = true;
}
break;
default:
}

View file

@ -151,6 +151,16 @@ describe('browser/settings.html', function() {
});
});
describe('Minimize to tray', function() {
it('should appear on darwin', function() {
const expected = (process.platform === 'darwin');
env.addClientCommands(this.app.client);
return this.app.client
.loadSettingsPage()
.isExisting('#inputMinimizeToTray').should.eventually.equal(expected)
});
});
describe('Toggle window visibility when clicking on the tray icon', function() {
it('should appear on win32', function() {
const expected = (process.platform === 'win32');