fixes for new battery options

JohnCoates · Oct 31, 2018 · a791b2b · a791b2b
1 parent ec5ca4a
commit a791b2b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/Aerial/Source/Controllers/PreferencesWindowController.swift b/Aerial/Source/Controllers/PreferencesWindowController.swift
@@ -578,11 +578,16 @@ class PreferencesWindowController: NSWindowController, NSOutlineViewDataSource,
     @IBAction func alternateVideoFormatChange(_ sender: NSPopUpButton) {
         debugLog("UI alternatePopupVideoFormat: \(sender.indexOfSelectedItem)")
         preferences.alternateVideoFormat = sender.indexOfSelectedItem
+        changeBatteryOverrideState(to: true)
     }
 
     func changeBatteryOverrideState(to: Bool) {
-        powerSavingOnLowBatteryCheckbox.isEnabled = to
         alternatePopupVideoFormat.isEnabled = to
+        if !to || (to && preferences.alternateVideoFormat != Preferences.AlternateVideoFormat.powerSaving.rawValue) {
+            powerSavingOnLowBatteryCheckbox.isEnabled = to
+        } else {
+            powerSavingOnLowBatteryCheckbox.isEnabled = false
+        }
     }
 
     @IBAction func popupVideoFormatChange(_ sender: NSPopUpButton) {

diff --git a/Aerial/Source/Views/AerialView.swift b/Aerial/Source/Views/AerialView.swift
@@ -173,9 +173,10 @@ class AerialView: ScreenSaverView {
         let preferences = Preferences.sharedInstance
         let timeManagement = TimeManagement.sharedInstance
 
-        if preferences.overrideOnBattery && timeManagement.isOnBattery() {
+        if preferences.overrideOnBattery && timeManagement.isOnBattery() && !isPreview {
             if preferences.alternateVideoFormat == Preferences.AlternateVideoFormat.powerSaving.rawValue ||
                 (preferences.powerSavingOnLowBattery && timeManagement.isBatteryLow()) {
+                isDisabled = true
                 timeManagement.setBrightness(level: 0.0)
                 return
             }