Minor Changes

Author: manosim

src/js/__tests__/components/settings-modal.js

@@ -136,13 +136,6 @@ describe('components/settings-modal.js', () => {
     expect(props.toggleSettingsModal).toHaveBeenCalledTimes(1);
   });
 
-  it('should close the modal pressing the close button', () => {
-    const wrapper = shallow(<SettingsModal {...props} />);
-    expect(wrapper).toBeDefined();
-    wrapper.find('.btn-secondary').simulate('click');
-    expect(props.toggleSettingsModal).toHaveBeenCalledTimes(1);
-  });
-
   it('should toggle the showOnlyParticipating checbox', () => {
     const wrapper = shallow(<SettingsModal {...props} />);
     expect(wrapper).toBeDefined();

src/js/components/logos/dark.js

@@ -9,7 +9,7 @@ export default class LogoWhite extends React.PureComponent {
   render() {
     /* eslint-disable */
     return (
-      <svg className="logo" onClick={() => this.props.onClick && this.props.onClick()} xmlns="http://www.w3.org/2000/svg" xmlnsXlink="http://www.w3.org/1999/xlink" viewBox="0 0 180.31 182.52">
+      <svg className={`logo ${this.props.className}`} onClick={() => this.props.onClick && this.props.onClick()} xmlns="http://www.w3.org/2000/svg" xmlnsXlink="http://www.w3.org/1999/xlink" viewBox="0 0 180.31 182.52">
         <defs>
 
           <linearGradient id="linear-dark-gradient" x1="300.53" y1="294.89" x2="130.22" y2="240.53" gradientUnits="userSpaceOnUse">

src/js/components/logos/white.js

@@ -9,7 +9,7 @@ export default class LogoWhite extends React.PureComponent {
   render() {
     /* eslint-disable */
     return (
-      <svg className="logo" onClick={() => this.props.onClick && this.props.onClick()} xmlns="http://www.w3.org/2000/svg" xmlnsXlink="http://www.w3.org/1999/xlink" viewBox="0 0 180.31 182.52">
+      <svg className={`logo ${this.props.className}`} onClick={() => this.props.onClick && this.props.onClick()} xmlns="http://www.w3.org/2000/svg" xmlnsXlink="http://www.w3.org/1999/xlink" viewBox="0 0 180.31 182.52">
         <defs>
 
           <linearGradient id="linear-white-gradient" x1="300.53" y1="294.89" x2="130.22" y2="240.53" gradientUnits="userSpaceOnUse">

src/js/components/settings-modal.js

@@ -130,11 +130,7 @@ export class SettingsModal extends React.Component {
 
           <div className="modal-footer">
             <button className="btn btn-outline-danger" onClick={() => this.logout()}>
-              <span className="octicon octicon-sign-out" /> Logout
-            </button>
-
-            <button className="btn btn-secondary" onClick={() => this.props.toggleSettingsModal()}>
-              Close
+              <span className="octicon octicon-sign-out" /> Logout from all accounts
             </button>
           </div>
         </div>

src/js/components/sidebar.js

@@ -68,7 +68,7 @@ export class Sidebar extends React.Component {
         <div
           key={idx}
           title={account.get('hostname')}
-          className={`badge-account${(enterpriseAccounts.size === idx + 1) && ' last'}`}
+          className={`badge-account${(enterpriseAccounts.size === idx + 1) ? ' last' : ''}`}
           >
           <div className="mr-auto name">{accountDomain}</div>
           <div>{notificationsCount === 0 ? <span className="octicon octicon-check" /> : notificationsCount}</div>

src/js/routes/login.js

@@ -23,7 +23,7 @@ export class LoginPage extends React.Component {
 
     return (
       <div className="container-fluid main-container login">
-        <LogoDark />
+        <LogoDark className="mt-5" />
 
         <div className="desc">GitHub Notifications<br />in your menu bar.</div>
         <button