From db017ae915f9e9ec71f959b08b42e5eb75d92fb8 Mon Sep 17 00:00:00 2001
From: Sam Johnson <sam@scj.me>
Date: Thu, 25 Apr 2024 22:04:16 -0500
Subject: [PATCH 1/3] Make inverted icons configurable

---
 RetroBar/Controls/NotifyIcon.xaml.cs          | 14 ++----
 RetroBar/Controls/NotifyIconList.xaml.cs      |  4 +-
 .../NotifyIconCanInvertConverter.cs           | 26 +++++++++++
 RetroBar/Extensions/NotifyIconExtensions.cs   | 44 +++++++++++++++++++
 RetroBar/RetroBar.csproj                      |  2 +-
 RetroBar/Utilities/Settings.cs                | 21 ++++++++-
 6 files changed, 96 insertions(+), 15 deletions(-)
 create mode 100644 RetroBar/Converters/NotifyIconCanInvertConverter.cs
 create mode 100644 RetroBar/Extensions/NotifyIconExtensions.cs

diff --git a/RetroBar/Controls/NotifyIcon.xaml.cs b/RetroBar/Controls/NotifyIcon.xaml.cs
index e7d6146b..844a05c9 100644
--- a/RetroBar/Controls/NotifyIcon.xaml.cs
+++ b/RetroBar/Controls/NotifyIcon.xaml.cs
@@ -6,6 +6,7 @@
 using ManagedShell.Common.Helpers;
 using ManagedShell.Interop;
 using ManagedShell.WindowsTray;
+using RetroBar.Extensions;
 using RetroBar.Utilities;
 
 namespace RetroBar.Controls
@@ -33,21 +34,12 @@ public NotifyIcon()
 
         private void applyEffects()
         {
-            if ((!EnvironmentHelper.IsWindows10OrBetter && Settings.Instance.InvertIconsMode == Settings.InvertIconsOption.WhenNeededByTheme) || TrayIcon == null || Settings.Instance.InvertIconsMode == Settings.InvertIconsOption.Never)
+            if (TrayIcon == null || Settings.Instance.InvertIconsMode == Settings.InvertIconsOption.Never)
             {
                 return;
             }
 
-            string iconGuid = TrayIcon.GUID.ToString();
-
-            if (!(iconGuid == NotificationArea.HARDWARE_GUID ||
-                iconGuid == NotificationArea.UPDATE_GUID ||
-                iconGuid == NotificationArea.MICROPHONE_GUID ||
-                iconGuid == NotificationArea.LOCATION_GUID ||
-                iconGuid == NotificationArea.MEETNOW_GUID ||
-                iconGuid == NotificationArea.NETWORK_GUID ||
-                iconGuid == NotificationArea.POWER_GUID ||
-                iconGuid == NotificationArea.VOLUME_GUID))
+            if (!TrayIcon.CanInvert())
             {
                 return;
             }
diff --git a/RetroBar/Controls/NotifyIconList.xaml.cs b/RetroBar/Controls/NotifyIconList.xaml.cs
index 92d8f431..634039fc 100644
--- a/RetroBar/Controls/NotifyIconList.xaml.cs
+++ b/RetroBar/Controls/NotifyIconList.xaml.cs
@@ -51,12 +51,12 @@ private void Settings_PropertyChanged(object sender, PropertyChangedEventArgs e)
                     NotifyIcons.ItemsSource = allNotifyIconsSource.View;
                 }
             }
-            else if (e.PropertyName == "InvertIconsMode")
+            else if (e.PropertyName == "InvertIconsMode" || e.PropertyName == "InvertNotifyIcons")
             {
                 // Reload icons
                 NotifyIcons.ItemsSource = null;
 
-                if (Settings.Instance.CollapseNotifyIcons)
+                if (Settings.Instance.CollapseNotifyIcons && NotifyIconToggleButton.IsChecked != true)
                 {
                     NotifyIcons.ItemsSource = pinnedNotifyIconsSource.View;
                 }
diff --git a/RetroBar/Converters/NotifyIconCanInvertConverter.cs b/RetroBar/Converters/NotifyIconCanInvertConverter.cs
new file mode 100644
index 00000000..78b27c70
--- /dev/null
+++ b/RetroBar/Converters/NotifyIconCanInvertConverter.cs
@@ -0,0 +1,26 @@
+using ManagedShell.WindowsTray;
+using RetroBar.Extensions;
+using System;
+using System.Windows.Data;
+
+namespace RetroBar.Converters
+{
+    [ValueConversion(typeof(NotifyIcon), typeof(bool))]
+    public class NotifyIconCanInvertConverter : IValueConverter
+    {
+        public object Convert(object value, Type targetType, object parameter, System.Globalization.CultureInfo culture)
+        {
+            if (value is NotifyIcon icon)
+            {
+                return icon.CanInvert();
+            }
+
+            return Binding.DoNothing;
+        }
+
+        public object ConvertBack(object value, Type targetType, object parameter, System.Globalization.CultureInfo culture)
+        {
+            return Binding.DoNothing;
+        }
+    }
+}
diff --git a/RetroBar/Extensions/NotifyIconExtensions.cs b/RetroBar/Extensions/NotifyIconExtensions.cs
new file mode 100644
index 00000000..ed886ec4
--- /dev/null
+++ b/RetroBar/Extensions/NotifyIconExtensions.cs
@@ -0,0 +1,44 @@
+using ManagedShell.WindowsTray;
+using RetroBar.Utilities;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+
+namespace RetroBar.Extensions
+{
+    public static class NotifyIconExtensions
+    {
+
+        public static string GetInvertIdentifier(this NotifyIcon icon)
+        {
+            if (icon.GUID != default) return icon.GUID.ToString();
+            else return icon.Path + ":" + icon.UID.ToString();
+        }
+
+        public static bool CanInvert(this NotifyIcon icon) {
+            return Settings.Instance.InvertNotifyIcons.Contains(icon.GetInvertIdentifier());
+        }
+
+        public static void SetCanInvert(this NotifyIcon icon, bool canInvert)
+        {
+            var identifier = icon.GetInvertIdentifier();
+            var settings = new List<string>(Settings.Instance.InvertNotifyIcons);
+            var changed = false;
+
+            if (!canInvert)
+            {
+                changed = settings.Remove(identifier);
+            }
+            else if (!settings.Contains(identifier))
+            {
+                settings.Add(identifier);
+                changed = true;
+            }
+
+            if (changed)
+            {
+                Settings.Instance.InvertNotifyIcons = settings;
+            }
+        }
+    }
+}
diff --git a/RetroBar/RetroBar.csproj b/RetroBar/RetroBar.csproj
index f2ddb47a..bc2733f0 100644
--- a/RetroBar/RetroBar.csproj
+++ b/RetroBar/RetroBar.csproj
@@ -45,7 +45,7 @@
 
   <ItemGroup>
     <PackageReference Include="gong-wpf-dragdrop" Version="3.1.1" />
-    <PackageReference Include="ManagedShell" Version="0.0.240" />
+    <PackageReference Include="ManagedShell" Version="0.0.245" />
     <PackageReference Include="System.Net.Http.Json" Version="6.0.0" />
   </ItemGroup>
 
diff --git a/RetroBar/Utilities/Settings.cs b/RetroBar/Utilities/Settings.cs
index b3d17607..dd3cdd52 100644
--- a/RetroBar/Utilities/Settings.cs
+++ b/RetroBar/Utilities/Settings.cs
@@ -1,4 +1,6 @@
 using ManagedShell.AppBar;
+using ManagedShell.Common.Helpers;
+using ManagedShell.WindowsTray;
 using System.Collections.Generic;
 using System.ComponentModel;
 using System.Runtime.CompilerServices;
@@ -221,6 +223,23 @@ public bool CollapseNotifyIcons
             }
         }
 
+        private List<string> _invertNotifyIcons = new List<string> { NotificationArea.HARDWARE_GUID, NotificationArea.UPDATE_GUID, NotificationArea.MICROPHONE_GUID, NotificationArea.LOCATION_GUID, NotificationArea.MEETNOW_GUID, NotificationArea.NETWORK_GUID, NotificationArea.POWER_GUID, NotificationArea.VOLUME_GUID };
+        public List<string> InvertNotifyIcons
+        {
+            get
+            {
+                return _invertNotifyIcons;
+            }
+            set
+            {
+                if (_invertNotifyIcons != value)
+                {
+                    _invertNotifyIcons = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
+
         private string[] _pinnedNotifyIcons = { "7820ae76-23e3-4229-82c1-e41cb67d5b9c", "7820ae75-23e3-4229-82c1-e41cb67d5b9c", "7820ae74-23e3-4229-82c1-e41cb67d5b9c", "7820ae73-23e3-4229-82c1-e41cb67d5b9c" };
         public string[] PinnedNotifyIcons
         {
@@ -426,7 +445,7 @@ public bool LockTaskbar
             }
         }
 
-        private InvertIconsOption _invertIconsMode = InvertIconsOption.WhenNeededByTheme;
+        private InvertIconsOption _invertIconsMode = EnvironmentHelper.IsWindows10OrBetter ? InvertIconsOption.WhenNeededByTheme : InvertIconsOption.Never;
         public InvertIconsOption InvertIconsMode
         {
             get

From 4eea032ec1d360646b75e67d73cc00d0913d0966 Mon Sep 17 00:00:00 2001
From: Sam Johnson <sam@scj.me>
Date: Sat, 27 Apr 2024 00:01:52 -0500
Subject: [PATCH 2/3] Add invert setting to customize notifications window

---
 RetroBar/Converters/BoolToIntConverter.cs     |  7 ++-
 RetroBar/Languages/English.xaml               |  1 +
 RetroBar/NotificationPropertiesWindow.xaml    | 22 +++++++-
 RetroBar/NotificationPropertiesWindow.xaml.cs | 56 ++++++++++++++++++-
 RetroBar/PropertiesWindow.xaml                |  3 +
 5 files changed, 84 insertions(+), 5 deletions(-)

diff --git a/RetroBar/Converters/BoolToIntConverter.cs b/RetroBar/Converters/BoolToIntConverter.cs
index b777b1da..e0906bcb 100644
--- a/RetroBar/Converters/BoolToIntConverter.cs
+++ b/RetroBar/Converters/BoolToIntConverter.cs
@@ -10,7 +10,12 @@ public object Convert(object value, Type targetType, object parameter, System.Gl
         {
             if (value is bool boolValue)
             {
-                return boolValue ? 1 : 0;
+                var multiplier = 1;
+                if (parameter is string strValue)
+                {
+                    multiplier = System.Convert.ToInt32(strValue);
+                }
+                return boolValue ? 1 * multiplier : 0;
             }
 
             return Binding.DoNothing;
diff --git a/RetroBar/Languages/English.xaml b/RetroBar/Languages/English.xaml
index 43bfdee5..e1d02a05 100644
--- a/RetroBar/Languages/English.xaml
+++ b/RetroBar/Languages/English.xaml
@@ -65,6 +65,7 @@
     <s:String x:Key="always_show">Always show</s:String>
     <s:String x:Key="name_heading">Name</s:String>
     <s:String x:Key="behavior_heading">Behavior</s:String>
+    <s:String x:Key="invert_heading">Invert</s:String>
 
     <s:String x:Key="start_text">Start</s:String>
     <s:String x:Key="start_text_xp">start</s:String>
diff --git a/RetroBar/NotificationPropertiesWindow.xaml b/RetroBar/NotificationPropertiesWindow.xaml
index 4df39564..0e928fdd 100644
--- a/RetroBar/NotificationPropertiesWindow.xaml
+++ b/RetroBar/NotificationPropertiesWindow.xaml
@@ -2,6 +2,7 @@
         xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
         xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
         xmlns:converters="clr-namespace:RetroBar.Converters"
+        xmlns:Settings="clr-namespace:RetroBar.Utilities"
         Title="{DynamicResource customize_notifications}"
         Height="406"
         Width="364"
@@ -12,7 +13,9 @@
     <Window.Resources>
         <ResourceDictionary>
             <converters:BoolToIntConverter x:Key="boolToIntConverter" />
+            <converters:BoolToVisibilityConverter x:Key="boolToVisibilityConverter" />
             <converters:NewLineToSpaceConverter x:Key="newLineToSpaceConverter" />
+            <converters:NotifyIconCanInvertConverter x:Key="notifyIconCanInvertConverter" />
             <Style TargetType="{x:Type ComboBox}">
                 <Setter Property="HorizontalAlignment"
                         Value="Right" />
@@ -54,11 +57,13 @@
         </Grid.RowDefinitions>
         <DockPanel Grid.Row="0">
             <TextBlock Text="{DynamicResource customize_notifications_info}"
+                       Visibility="{Binding Source={x:Static Settings:Settings.Instance}, Path=CollapseNotifyIcons, UpdateSourceTrigger=PropertyChanged, Converter={StaticResource boolToVisibilityConverter}}"
                        TextWrapping="Wrap"
+                       Margin="0,0,0,20"
                        DockPanel.Dock="Top" />
             <TextBlock Text="{DynamicResource customize_notifications_instruction}"
                        TextWrapping="Wrap"
-                       Margin="0,20,0,3"
+                       Margin="0,0,0,3"
                        DockPanel.Dock="Top" />
             <ListView VerticalAlignment="Stretch"
                       ItemsSource="{Binding AllIcons}"
@@ -66,11 +71,12 @@
                 <ListView.View>
                     <GridView>
                         <GridViewColumn Header="{DynamicResource name_heading}"
-                                        Width="180">
+                                        Width="145">
                             <GridViewColumn.CellTemplate>
                                 <DataTemplate>
                                     <DockPanel>
                                         <Image Source="{Binding Icon}"
+                                               Loaded="Icon_Loaded"
                                                Width="16"
                                                Height="16"
                                                Margin="0,0,3,0"
@@ -85,7 +91,7 @@
                             </GridViewColumn.CellTemplate>
                         </GridViewColumn>
                         <GridViewColumn Header="{DynamicResource behavior_heading}"
-                                        Width="120">
+                                        Width="{Binding Source={x:Static Settings:Settings.Instance}, Path=CollapseNotifyIcons, UpdateSourceTrigger=PropertyChanged, Converter={StaticResource boolToIntConverter}, ConverterParameter=120}">
                             <GridViewColumn.CellTemplate>
                                 <DataTemplate>
                                     <StackPanel>
@@ -128,6 +134,16 @@
                                 </DataTemplate>
                             </GridViewColumn.CellTemplate>
                         </GridViewColumn>
+                        <GridViewColumn Header="{DynamicResource invert_heading}"
+                                        Width="Auto">
+                            <GridViewColumn.CellTemplate>
+                                <DataTemplate>
+                                    <CheckBox IsChecked="{Binding Mode=OneWay, Converter={StaticResource notifyIconCanInvertConverter}}"
+                                              Checked="InvertCheckBox_Checked"
+                                              Unchecked="InvertCheckBox_Unchecked" />
+                                </DataTemplate>
+                            </GridViewColumn.CellTemplate>
+                        </GridViewColumn>
                     </GridView>
                 </ListView.View>
             </ListView>
diff --git a/RetroBar/NotificationPropertiesWindow.xaml.cs b/RetroBar/NotificationPropertiesWindow.xaml.cs
index 8bdba465..86957b17 100644
--- a/RetroBar/NotificationPropertiesWindow.xaml.cs
+++ b/RetroBar/NotificationPropertiesWindow.xaml.cs
@@ -1,6 +1,8 @@
 using ManagedShell.WindowsTray;
+using RetroBar.Extensions;
 using RetroBar.Utilities;
 using System.Windows;
+using System.Windows.Controls;
 
 namespace RetroBar
 {
@@ -20,6 +22,17 @@ private NotificationPropertiesWindow(NotificationArea notificationArea)
             InitializeComponent();
 
             DataContext = _notificationArea;
+            Settings.Instance.PropertyChanged += Settings_PropertyChanged;
+        }
+
+        private void Settings_PropertyChanged(object sender, System.ComponentModel.PropertyChangedEventArgs e)
+        {
+            if (e.PropertyName == "InvertNotifyIcons")
+            {
+                // Reload icons
+                DataContext = null;
+                DataContext = _notificationArea;
+            }
         }
 
         public static void Open(NotificationArea notificationArea, Point position)
@@ -44,12 +57,53 @@ private void OK_OnClick(object sender, RoutedEventArgs e)
 
         private void Window_Closing(object sender, System.ComponentModel.CancelEventArgs e)
         {
+            Settings.Instance.PropertyChanged -= Settings_PropertyChanged;
             _instance = null;
         }
 
-        private void BehaviorComboBox_SelectionChanged(object sender, System.Windows.Controls.SelectionChangedEventArgs e)
+        private void BehaviorComboBox_SelectionChanged(object sender, SelectionChangedEventArgs e)
         {
             Settings.Instance.PinnedNotifyIcons = _notificationArea.PinnedNotifyIcons;
         }
+
+        private void InvertCheckBox_Checked(object sender, RoutedEventArgs e)
+        {
+            var checkBox = sender as CheckBox;
+            var icon = checkBox.DataContext as NotifyIcon;
+            icon.SetCanInvert(true);
+        }
+
+        private void InvertCheckBox_Unchecked(object sender, RoutedEventArgs e)
+        {
+            var checkBox = sender as CheckBox;
+            var icon = checkBox.DataContext as NotifyIcon;
+            icon.SetCanInvert(false);
+        }
+
+        private void Icon_Loaded(object sender, RoutedEventArgs e)
+        {
+            var image = sender as Image;
+            var notifyIcon = image.DataContext as NotifyIcon;
+            applyEffects(image, notifyIcon);
+        }
+
+        private void applyEffects(Image NotifyIconImage, NotifyIcon TrayIcon)
+        {
+            if (TrayIcon == null)
+            {
+                return;
+            }
+
+            if (!TrayIcon.CanInvert())
+            {
+                NotifyIconImage.Effect = null;
+                return;
+            }
+
+            if (NotifyIconImage.Effect == null)
+            {
+                NotifyIconImage.Effect = new InvertEffect();
+            }
+        }
     }
 }
diff --git a/RetroBar/PropertiesWindow.xaml b/RetroBar/PropertiesWindow.xaml
index 60101eb4..31520a82 100644
--- a/RetroBar/PropertiesWindow.xaml
+++ b/RetroBar/PropertiesWindow.xaml
@@ -334,6 +334,9 @@
                                       SelectedIndex="{Binding Source={x:Static Settings:Settings.Instance}, Path=InvertIconsMode, UpdateSourceTrigger=PropertyChanged, Converter={StaticResource enumConverter}}"
                                       SelectionChanged="cboInvertIconsMode_SelectionChanged" />
                         </DockPanel>
+                        <Button HorizontalAlignment="Right"
+                                        Content="{DynamicResource customize}"
+                                        Click="CustomizeNotifications_OnClick" />
                     </StackPanel>
                     <GroupBox Header="{DynamicResource multiple_displays}"
                               Margin="0,0,0,10">

From 801e64aa23748b1ee147107f01ade5be0a9d7ccf Mon Sep 17 00:00:00 2001
From: Sam Johnson <sam@scj.me>
Date: Sat, 27 Apr 2024 00:18:34 -0500
Subject: [PATCH 3/3] Remove unnecessary usings

---
 RetroBar/Extensions/NotifyIconExtensions.cs | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/RetroBar/Extensions/NotifyIconExtensions.cs b/RetroBar/Extensions/NotifyIconExtensions.cs
index ed886ec4..99c71039 100644
--- a/RetroBar/Extensions/NotifyIconExtensions.cs
+++ b/RetroBar/Extensions/NotifyIconExtensions.cs
@@ -1,14 +1,11 @@
 using ManagedShell.WindowsTray;
 using RetroBar.Utilities;
-using System;
 using System.Collections.Generic;
-using System.Linq;
 
 namespace RetroBar.Extensions
 {
     public static class NotifyIconExtensions
     {
-
         public static string GetInvertIdentifier(this NotifyIcon icon)
         {
             if (icon.GUID != default) return icon.GUID.ToString();