diff --git a/src/browser/components/SettingsPage.jsx b/src/browser/components/SettingsPage.jsx
index 488115b0..0eaeeb2d 100644
--- a/src/browser/components/SettingsPage.jsx
+++ b/src/browser/components/SettingsPage.jsx
@@ -1,5 +1,5 @@
const React = require('react');
-const {Col, Grid, Navbar, Input, Row} = require('react-bootstrap');
+const {Button, Checkbox, Col, FormGroup, Grid, Navbar, Row} = require('react-bootstrap');
const {ipcRenderer, remote} = require('electron');
const AutoLaunch = require('auto-launch');
@@ -92,16 +92,16 @@ const SettingsPage = React.createClass({
},
handleChangeDisableWebSecurity() {
this.setState({
- disablewebsecurity: this.refs.disablewebsecurity.getChecked()
+ disablewebsecurity: !this.refs.disablewebsecurity.props.checked
});
},
handleChangeHideMenuBar() {
this.setState({
- hideMenuBar: this.refs.hideMenuBar.getChecked()
+ hideMenuBar: !this.refs.hideMenuBar.props.checked
});
},
handleChangeShowTrayIcon() {
- var shouldShowTrayIcon = this.refs.showTrayIcon.getChecked();
+ var shouldShowTrayIcon = !this.refs.showTrayIcon.props.checked;
this.setState({
showTrayIcon: shouldShowTrayIcon
});
@@ -114,18 +114,18 @@ const SettingsPage = React.createClass({
},
handleChangeTrayIconTheme() {
this.setState({
- trayIconTheme: this.refs.trayIconTheme.getValue()
+ trayIconTheme: !this.refs.trayIconTheme.props.checked
});
},
handleChangeAutoStart() {
this.setState({
- autostart: this.refs.autostart.getChecked()
+ autostart: !this.refs.autostart.props.checked
});
},
handleChangeMinimizeToTray() {
var shouldMinimizeToTray =
- (process.platform !== 'darwin' || this.refs.showTrayIcon.getChecked()) &&
- this.refs.minimizeToTray.getChecked();
+ (process.platform !== 'darwin' || !this.refs.showTrayIcon.props.checked) &&
+ !this.refs.minimizeToTray.props.checked;
this.setState({
minimizeToTray: shouldMinimizeToTray
@@ -133,7 +133,7 @@ const SettingsPage = React.createClass({
},
handleChangeToggleWindowOnTrayIconClick() {
this.setState({
- toggleWindowOnTrayIconClick: this.refs.toggleWindowOnTrayIconClick.getChecked()
+ toggleWindowOnTrayIconClick: !this.refs.toggleWindowOnTrayIconClick.props.checked
});
},
toggleShowTeamForm() {
@@ -144,13 +144,13 @@ const SettingsPage = React.createClass({
handleFlashWindow() {
this.setState({
notifications: {
- flashWindow: this.refs.flashWindow.getChecked() ? 2 : 0
+ flashWindow: this.refs.flashWindow.props.checked ? 0 : 2
}
});
},
handleShowUnreadBadge() {
this.setState({
- showUnreadBadge: this.refs.showUnreadBadge.getChecked()
+ showUnreadBadge: !this.refs.showUnreadBadge.props.checked
});
},
render() {
@@ -169,120 +169,103 @@ const SettingsPage = React.createClass({
var options = [];
if (process.platform === 'win32' || process.platform === 'linux') {
options.push(
- );
+ >{'Hide menu bar (Press Alt to show menu bar)'});
}
if (process.platform === 'darwin' || process.platform === 'linux') {
options.push(
- );
+ >{process.platform === 'darwin' ?
+ 'Show icon on menu bar (need to restart the application)' :
+ 'Show icon in notification area (need to restart the application)'});
}
if (process.platform === 'linux') {
options.push(
-
+ >{'Icon theme (Need to restart the application)'}
- );
+ );
}
options.push(
- );
+ >{'Allow mixed content (Enabling allows both secure and insecure content, images and scripts to render and execute. Disabling allows only secure content.)'});
//OSX has an option in the Dock, to set the app to autostart, so we choose to not support this option for OSX
if (process.platform === 'win32' || process.platform === 'linux') {
options.push(
- );
+ >{'Start app on login.'});
}
if (process.platform === 'darwin' || process.platform === 'linux') {
options.push(
- );
+ >{this.state.trayWasVisible || !this.state.showTrayIcon ? 'Leave app running in notification area when the window is closed' : 'Leave app running in notification area when the window is closed (available on next restart)'});
}
if (process.platform === 'win32') {
options.push(
- );
+ >{'Toggle window visibility when clicking on the tray icon.'});
}
if (process.platform === 'darwin' || process.platform === 'win32') {
options.push(
- );
+ >{'Show red badge on taskbar icon to indicate unread messages. Regardless of this setting, mentions are always indicated with a red badge and item count on the taskbar icon.'});
}
if (process.platform === 'win32' || process.platform === 'linux') {
options.push(
- );
+ >{'Flash the taskbar icon when a new message is received.'});
}
const settingsPage = {
@@ -323,7 +306,11 @@ const SettingsPage = React.createClass({
{'App options'}
- { options }
+ { options.map((opt, i) => (
+
+ {opt}
+
+ )) }
) : null;
@@ -377,19 +364,19 @@ const SettingsPage = React.createClass({
className='text-right'
style={settingsPage.footer}
>
-
+ >{'Cancel'}
{ ' ' }
-
+ >{'Save'}
diff --git a/src/browser/components/TeamList.jsx b/src/browser/components/TeamList.jsx
index 1fb40451..fa64a014 100644
--- a/src/browser/components/TeamList.jsx
+++ b/src/browser/components/TeamList.jsx
@@ -96,7 +96,7 @@ const TeamList = React.createClass({
}
return (
-
+
{ teamNodes }
{ addTeamForm }
diff --git a/src/package.json b/src/package.json
index c1fb167d..dc2dac09 100644
--- a/src/package.json
+++ b/src/package.json
@@ -21,7 +21,7 @@
"electron-squirrel-startup": "^1.0.0",
"os-locale": "^1.4.0",
"react": "^15.3.0",
- "react-bootstrap": "~0.29.0",
+ "react-bootstrap": "~0.30.6",
"react-dom": "^15.3.0",
"yargs": "^3.32.0"
}
diff --git a/test/specs/browser/settings_test.js b/test/specs/browser/settings_test.js
index ff1e07a5..0859abcc 100644
--- a/test/specs/browser/settings_test.js
+++ b/test/specs/browser/settings_test.js
@@ -65,9 +65,9 @@ describe('browser/settings.html', function desc() {
return this.app.client.
loadSettingsPage().
scroll('#inputHideMenuBar').
- isSelected('#inputHideMenuBar input').then((isSelected) => {
+ isSelected('#inputHideMenuBar').then((isSelected) => {
if (isSelected !== v) {
- return this.app.client.click('#inputHideMenuBar input');
+ return this.app.client.click('#inputHideMenuBar');
}
return true;
}).
@@ -83,7 +83,7 @@ describe('browser/settings.html', function desc() {
return this.app.client. // confirm actual behavior
browserWindow.isMenuBarAutoHide().should.eventually.equal(v).
loadSettingsPage().
- isSelected('#inputHideMenuBar input').should.eventually.equal(v);
+ isSelected('#inputHideMenuBar').should.eventually.equal(v);
});
});
});
@@ -98,9 +98,9 @@ describe('browser/settings.html', function desc() {
return this.app.client.
loadSettingsPage().
scroll('#inputDisableWebSecurity').
- isSelected('#inputDisableWebSecurity input').then((isSelected) => {
+ isSelected('#inputDisableWebSecurity').then((isSelected) => {
if (isSelected !== v) {
- return this.app.client.click('#inputDisableWebSecurity input');
+ return this.app.client.click('#inputDisableWebSecurity');
}
return true;
}).
@@ -125,7 +125,7 @@ describe('browser/settings.html', function desc() {
});
}).
loadSettingsPage().
- isSelected('#inputDisableWebSecurity input').should.eventually.equal(v);
+ isSelected('#inputDisableWebSecurity').should.eventually.equal(v);
});
});
});