From 174b8df73828aa6ef1464e0ca49cba07646965e8 Mon Sep 17 00:00:00 2001
From: Guillaume Louel <guillaume@louel.fr>
Date: Sat, 3 Nov 2018 13:18:06 +0100
Subject: [PATCH] Last fixes for 1.4.5

---
 .../PreferencesWindowController.swift         | 20 +++++++++++++++++++
 Resources/PreferencesWindow.xib               | 11 ++++++----
 2 files changed, 27 insertions(+), 4 deletions(-)

diff --git a/Aerial/Source/Controllers/PreferencesWindowController.swift b/Aerial/Source/Controllers/PreferencesWindowController.swift
index 953ce773..649167b1 100644
--- a/Aerial/Source/Controllers/PreferencesWindowController.swift
+++ b/Aerial/Source/Controllers/PreferencesWindowController.swift
@@ -418,6 +418,10 @@ class PreferencesWindowController: NSWindowController, NSOutlineViewDataSource,
             if preferences.darkModeNightOverride {
                 overrideNightOnDarkMode.state = .on
             }
+            // We disable the checkbox if we are on nightShift mode
+            if preferences.timeMode == Preferences.TimeMode.lightDarkMode.rawValue {
+                overrideNightOnDarkMode.isEnabled = false
+            }
         } else {
             overrideNightOnDarkMode.isEnabled = false
         }
@@ -1039,6 +1043,12 @@ class PreferencesWindowController: NSWindowController, NSOutlineViewDataSource,
 
     // MARK: - Time panel
 
+    @IBAction func overrideNightOnDarkModeClick(_ button: NSButton) {
+        let onState = (button.state == NSControl.StateValue.on)
+        preferences.darkModeNightOverride = onState
+        debugLog("UI overrideNightDarkMode: \(onState)")
+    }
+
     @IBAction func enterCoordinatesButtonClick(_ sender: Any) {
         if enterCoordinatesPanel.isVisible {
             enterCoordinatesPanel.close()
@@ -1052,6 +1062,16 @@ class PreferencesWindowController: NSWindowController, NSOutlineViewDataSource,
     }
 
     @IBAction func timeModeChange(_ sender: NSButton?) {
+        debugLog("UI timeModeChange")
+        if sender == timeLightDarkModeRadio {
+            print("dis")
+            overrideNightOnDarkMode.isEnabled = false
+        } else {
+            if #available(OSX 10.14, *) {
+                overrideNightOnDarkMode.isEnabled = true
+            }
+        }
+
         if sender == timeDisabledRadio {
             preferences.timeMode = Preferences.TimeMode.disabled.rawValue
         } else if sender == timeNightShiftRadio {
diff --git a/Resources/PreferencesWindow.xib b/Resources/PreferencesWindow.xib
index a54a0684..15ad328c 100644
--- a/Resources/PreferencesWindow.xib
+++ b/Resources/PreferencesWindow.xib
@@ -904,7 +904,7 @@ should appear</string>
                                         </view>
                                     </tabViewItem>
                                     <tabViewItem label="Time" identifier="" id="PtY-yV-jgj">
-                                        <view key="view" id="ziy-Hg-uzO">
+                                        <view key="view" ambiguous="YES" id="ziy-Hg-uzO">
                                             <rect key="frame" x="10" y="33" width="614" height="381"/>
                                             <autoresizingMask key="autoresizingMask" widthSizable="YES" heightSizable="YES"/>
                                             <subviews>
@@ -1149,12 +1149,15 @@ Shift, but macOS 10.12.4 or above and a compatible Mac are required) </string>
                                                     <imageCell key="cell" refusesFirstResponder="YES" alignment="left" imageScaling="proportionallyDown" image="NSHomeTemplate" id="lQX-Ux-Sz8"/>
                                                 </imageView>
                                                 <button toolTip="This will override the time settings to always show night videos if you have enabled dark mode" verticalHuggingPriority="750" fixedFrame="YES" translatesAutoresizingMaskIntoConstraints="NO" id="YJt-Nf-qyK">
-                                                    <rect key="frame" x="8" y="7" width="264" height="18"/>
+                                                    <rect key="frame" x="8" y="7" width="246" height="18"/>
                                                     <autoresizingMask key="autoresizingMask" flexibleMaxX="YES" flexibleMaxY="YES"/>
-                                                    <buttonCell key="cell" type="check" title="Always show night videos on dark mode" bezelStyle="regularSquare" imagePosition="left" inset="2" id="nZy-k8-5B4">
+                                                    <buttonCell key="cell" type="check" title="Show only night videos in Dark Mode" bezelStyle="regularSquare" imagePosition="left" inset="2" id="nZy-k8-5B4">
                                                         <behavior key="behavior" changeContents="YES" doesNotDimImage="YES" lightByContents="YES"/>
                                                         <font key="font" metaFont="system"/>
                                                     </buttonCell>
+                                                    <connections>
+                                                        <action selector="overrideNightOnDarkModeClick:" target="-2" id="PVR-uW-k3k"/>
+                                                    </connections>
                                                 </button>
                                                 <button verticalHuggingPriority="750" fixedFrame="YES" translatesAutoresizingMaskIntoConstraints="NO" id="a7p-Yo-yfa">
                                                     <rect key="frame" x="403" y="338" width="207" height="32"/>
@@ -1437,7 +1440,7 @@ Shift, but macOS 10.12.4 or above and a compatible Mac are required) </string>
                                         </view>
                                     </tabViewItem>
                                     <tabViewItem label="Advanced" identifier="" id="asf-lu-1rA">
-                                        <view key="view" ambiguous="YES" id="303-Jh-DVp">
+                                        <view key="view" id="303-Jh-DVp">
                                             <rect key="frame" x="10" y="33" width="614" height="381"/>
                                             <autoresizingMask key="autoresizingMask" widthSizable="YES" heightSizable="YES"/>
                                             <subviews>