Fix scheduled theme change timing

Author: gregueiras

browser/main/Main.js

@@ -190,6 +190,7 @@ class Main extends React.Component {
       'menubar:togglemenubar',
       this.toggleMenuBarVisible.bind(this)
     )
+    clearInterval(this.refreshTheme)
   }
 
   toggleMenuBarVisible() {

browser/main/lib/ThemeManager.js

@@ -10,8 +10,8 @@ const chooseTheme = ui => {
   const minutes = now.getHours() * 60 + now.getMinutes()
 
   const isEndAfterStart = end > start
-  const isBetweenStartAndEnd = minutes >= start && minutes <= end
-  const isBetweenEndAndStart = minutes >= start || minutes <= end
+  const isBetweenStartAndEnd = minutes >= start && minutes < end
+  const isBetweenEndAndStart = minutes >= start || minutes < end
 
   if (
     (isEndAfterStart && isBetweenStartAndEnd) ||

browser/main/modals/PreferencesModal/UiTab.js

@@ -335,7 +335,7 @@ class UiTab extends React.Component {
           </div>
           <div styleName='group-section'>
             <div styleName='container'>
-              <div styleName='range-slider' id='firstRow'>
+              <div id='firstRow'>
                 <span
                   id='rs-bullet-1'
                   styleName='rs-label'
@@ -353,7 +353,7 @@ class UiTab extends React.Component {
                   onChange={e => this.handleUIChange(e)}
                 />
               </div>
-              <div styleName='range-slider' id='secondRow'>
+              <div id='secondRow'>
                 <span
                   id='rs-bullet-2'
                   styleName='rs-label'