Moved API call, added OS version check

Author: mansellan

Rubberduck.Core/Common/IOperatingSystem.cs

@@ -3,5 +3,6 @@
     public interface IOperatingSystem
     {
         void ShowFolder(string folderPath);
+        WindowsVersion? GetOSVersion();
     }
 }

Rubberduck.Core/Common/WinAPI/SHCore.cs

@@ -0,0 +1,41 @@
+using System.Runtime.InteropServices;
+
+namespace Rubberduck.Common.WinAPI
+{
+    public static class SHCore
+    {
+        /// <summary>
+        /// Sets the DPI awerness level of the current process.
+        /// </summary>
+        /// <param name="awareness">DPI awareness level.</param>
+        /// <returns>HRESULT of S_OK, E_INVALIDARG or E_ACCESSDENIED.</returns>
+        /// <remarks>
+        /// Only the first DPI awareness call made by a process will have effect, subsequent calls are disregarded.
+        /// Thus, calling this method before WPF loads will override the default WPF DPI awareness behavior.
+        /// </remarks>
+        [DllImport("SHCore.dll", SetLastError = true)]
+        public static extern bool SetProcessDpiAwareness(PROCESS_DPI_AWARENESS awareness);
+    }
+
+    /// <summary>
+    /// Describes DPI awareness of a process.
+    /// </summary>
+    public enum PROCESS_DPI_AWARENESS
+    {
+        /// <summary>
+        /// Process is not DPI aware. 
+        /// </summary>
+        Process_DPI_Unaware = 0,
+
+        /// <summary>
+        /// Process is aware of the System DPI (monitor 1).
+        /// </summary>
+        Process_System_DPI_Aware = 1,
+
+        /// <summary>
+        /// Process is aware of the DPI of individual monitors.
+        /// </summary>
+        Process_Per_Monitor_DPI_Aware = 2
+    }
+
+}

Rubberduck.Core/Common/WindowsOperatingSystem.cs

@@ -1,18 +1,56 @@
-using System.Diagnostics;
+using System;
+using System.Diagnostics;
 using System.IO;
+using System.Management;
+using System.Runtime.CompilerServices;
+using NLog;
 
 namespace Rubberduck.Common
 {
     public sealed class WindowsOperatingSystem : IOperatingSystem
     {
+        public static readonly ILogger _Logger = LogManager.GetCurrentClassLogger(); 
+
         public void ShowFolder(string folderPath)
         {
             if (!Directory.Exists(folderPath))
             {
                 Directory.CreateDirectory(folderPath);
             }
 
-            using (Process.Start(folderPath)) { }
+            using (Process.Start(folderPath))
+            {
+            }
+        }
+
+
+        public WindowsVersion? GetOSVersion()
+        {
+            try
+            {
+                var wmiEnum = new ManagementObjectSearcher("root\\CIMV2", "SELECT Version FROM  Win32_OperatingSystem")
+                    .Get().GetEnumerator();
+                wmiEnum.MoveNext();
+                var versionString = wmiEnum.Current.Properties["Version"].Value as string;
+
+                var versionElements = versionString?.Split('.');
+
+                if (versionElements?.Length >= 3 &&
+                    int.TryParse(versionElements[0], out var major) &&
+                    int.TryParse(versionElements[1], out var minor) &&
+                    int.TryParse(versionElements[2], out var build))
+                {
+                    return new WindowsVersion(major, minor, build);
+                }                
+            }
+            catch (Exception exception)
+            {
+                _Logger.Warn(exception, "Unable to determine OS Version");
+                return null;
+            }
+            _Logger.Warn("Unable to determine OS Version");
+            return null;
         }
     }
 }
+

Rubberduck.Core/Common/WindowsVersion.cs

@@ -0,0 +1,79 @@
+using System;
+
+namespace Rubberduck.Common
+{
+    public struct WindowsVersion : IComparable<WindowsVersion>
+    {
+        public static WindowsVersion Windows10_1809 = new WindowsVersion(10, 0, 17763);
+        public static WindowsVersion Windows10_1803 = new WindowsVersion(10, 0, 17134);
+        public static WindowsVersion Windows10_1709 = new WindowsVersion(10, 0, 16299);
+        public static WindowsVersion Windows10_1703 = new WindowsVersion(10, 0, 15063);
+        public static WindowsVersion Windows10_1607 = new WindowsVersion(10, 0, 14393);
+        public static WindowsVersion Windows10_1511 = new WindowsVersion(10, 0, 10586);
+        public static WindowsVersion Windows10 = new WindowsVersion(10, 0, 10240);
+        public static WindowsVersion Windows81_U1 = new WindowsVersion(6, 3, 9600);
+        public static WindowsVersion Windows81 = new WindowsVersion(6, 3, 9200);
+        public static WindowsVersion Windows8 = new WindowsVersion(6, 2, 9200);
+        public static WindowsVersion Windows7_SP1 = new WindowsVersion(6, 1, 7601);
+        public static WindowsVersion Windows7 = new WindowsVersion(6, 1, 7600);
+        public static WindowsVersion WindowsVista_SP2 = new WindowsVersion(6, 0, 6002);
+        public static WindowsVersion WindowsVista_SP1 = new WindowsVersion(6, 0, 6001);
+        public static WindowsVersion WindowsVista = new WindowsVersion(6, 0, 6000);
+
+        public WindowsVersion(int major, int minor, int build)
+        {
+            Major = major;
+            Minor = minor;
+            Build = build;
+        }
+
+        public int Major { get; }
+        public int Minor { get; }
+        public int Build { get; }
+
+        public int CompareTo(WindowsVersion other)
+        {
+            var majorComparison = Major.CompareTo(other.Major);
+            if (majorComparison != 0)
+            {
+                return majorComparison;
+            }
+
+            var minorComparison = Minor.CompareTo(other.Minor);
+
+            return minorComparison != 0
+                ? minorComparison
+                : Build.CompareTo(other.Build);
+        }
+
+        public static bool operator ==(WindowsVersion os1, WindowsVersion os2)
+        {
+            return os1.CompareTo(os2) == 0;
+        }
+
+        public static bool operator !=(WindowsVersion os1, WindowsVersion os2)
+        {
+            return os1.CompareTo(os2) != 0;
+        }
+
+        public static bool operator <(WindowsVersion os1, WindowsVersion os2)
+        {
+            return os1.CompareTo(os2) < 0;
+        }
+
+        public static bool operator >(WindowsVersion os1, WindowsVersion os2)
+        {
+            return os1.CompareTo(os2) > 0;
+        }
+
+        public static bool operator <=(WindowsVersion os1, WindowsVersion os2)
+        {
+            return os1.CompareTo(os2) <= 0;
+        }
+
+        public static bool operator >=(WindowsVersion os1, WindowsVersion os2)
+        {
+            return os1.CompareTo(os2) >= 0;
+        }
+    }
+}

Rubberduck.Core/Rubberduck.Core.csproj

@@ -29,6 +29,7 @@
     <Reference Include="PresentationCore" />
     <Reference Include="PresentationFramework" />
     <Reference Include="PresentationFramework.Aero" />
+    <Reference Include="System.Management" />
     <Reference Include="System.Net.Http" />
     <Reference Include="System.Windows.Forms" />
     <Reference Include="System.Xaml" />

Rubberduck.Core/UI/Settings/GeneralSettings.xaml

@@ -132,7 +132,7 @@
                     <CheckBox Margin="5,0,0,5" Content="{Resx ResxName=Rubberduck.Resources.RubberduckUI, Key=GeneralSettings_CompileBeforeParse}"
                               IsChecked="{Binding CompileBeforeParse}" />
                     <CheckBox Margin="5,0,0,5" Content="{Resx ResxName=Rubberduck.Resources.RubberduckUI, Key=GeneralSettings_SetDpiUnaware}"
-                              IsChecked="{Binding SetDpiUnaware}" />
+                              IsEnabled="{Binding SetDpiUnawareEnabled}"  IsChecked="{Binding SetDpiUnaware}" />
                     <StackPanel Orientation="Horizontal"/>
 
                     <Label Content="{Resx ResxName=Rubberduck.Resources.RubberduckUI, Key=GeneralSettings_MinimumLogLevelLabel}" FontWeight="SemiBold" />

Rubberduck.Core/UI/Settings/GeneralSettingsViewModel.cs

@@ -172,6 +172,15 @@ public bool SetDpiUnaware
             }
         }
 
+        public bool SetDpiUnawareEnabled
+        {
+            get
+            {
+                var osVersion = _operatingSystem.GetOSVersion();
+                return osVersion != null && osVersion >= WindowsVersion.Windows81;
+            }
+        }
+
         private bool SynchronizeVBESettings()
         {
             if (!_messageBox.ConfirmYesNo(RubberduckUI.GeneralSettings_CompileBeforeParse_WarnCompileOnDemandEnabled,

Rubberduck.Main/Extension.cs

@@ -12,6 +12,7 @@
 using System.Windows.Threading;
 using Castle.Windsor;
 using NLog;
+using Rubberduck.Common.WinAPI;
 using Rubberduck.Root;
 using Rubberduck.Resources;
 using Rubberduck.Resources.Registration;
@@ -164,7 +165,7 @@ private void InitializeAddIn()
                 {
                     if (_initialSettings.SetDpiUnaware)
                     {
-                        SetProcessDpiAwareness(PROCESS_DPI_AWARENESS.Process_DPI_Unaware);
+                        SHCore.SetProcessDpiAwareness(PROCESS_DPI_AWARENESS.Process_DPI_Unaware);
                     }
                 }
                 catch (Exception)
@@ -311,15 +312,5 @@ private void ShutdownAddIn()
                 }
             }
         }
-
-        [DllImport("SHCore.dll", SetLastError = true)]
-        private static extern bool SetProcessDpiAwareness(PROCESS_DPI_AWARENESS awareness);
-
-        private enum PROCESS_DPI_AWARENESS
-        {
-            Process_DPI_Unaware = 0,
-            Process_System_DPI_Aware = 1,
-            Process_Per_Monitor_DPI_Aware = 2
-        }
     }
 }