Bläddra i källkod

Make windows DPI aware to display properly on high-resolution screens. (#1983)

* Make Windows DPI aware to display properly on high-resolution screens.

* remove empty line

* Don't use app manifest, set process dpi aware programatically.

Store variables in Program.cs for use instead of re-creating them per class/ method.

* Fix for linux/osx

* Add braces

* Re-use manifest. It appears to be required on linux.

* Undo previous commit -- it appears linux was simply never affected.

* Addressed AcK's comments

* Remove unused usings

* Address comments by AcK #2

* Re-order

* Move FromHwnd call to ForceDpiAware class. Wrap in Try-Catch to prevent crashes on systems that don't support it.

* Additional code cleanup

* Remove "global::" reference.
sharmander 5 år sedan
förälder
incheckning
d5081e3f93

+ 1 - 0
Ryujinx.Common/Ryujinx.Common.csproj

@@ -7,6 +7,7 @@
 
   <ItemGroup>
     <PackageReference Include="MsgPack.Cli" Version="1.0.1" />
+    <PackageReference Include="System.Drawing.Common" Version="5.0.1" />
     <PackageReference Include="System.Management" Version="5.0.0" />
   </ItemGroup>
 

+ 45 - 0
Ryujinx.Common/System/ForceDpiAware.cs

@@ -0,0 +1,45 @@
+using Ryujinx.Common.Logging;
+using System;
+using System.Drawing;
+using System.Runtime.InteropServices;
+
+namespace Ryujinx.Common.System
+{
+    public static class ForceDpiAware
+    {
+        [DllImport("user32.dll")]
+        private static extern bool SetProcessDPIAware();
+
+        private static readonly double _standardDpiScale = 96.0;
+        private static readonly double _maxScaleFactor   = 1.25;
+
+        /// <summary>
+        /// Marks the application as DPI-Aware when running on the Windows operating system.
+        /// </summary>
+        public static void Windows()
+        {
+            // Make process DPI aware for proper window sizing on high-res screens.
+            if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && Environment.OSVersion.Version.Major >= 6)
+            {
+                SetProcessDPIAware();
+            }
+        }
+
+        public static double GetWindowScaleFactor()
+        {
+            double userDpiScale;
+
+            try
+            {
+                userDpiScale = Graphics.FromHwnd(IntPtr.Zero).DpiX;
+            }
+            catch (Exception e)
+            {
+                Logger.Warning?.Print(LogClass.Application, $"Couldn't determine monitor DPI: {e.Message}");
+                userDpiScale = 96.0;
+            }
+
+            return Math.Min(userDpiScale / _standardDpiScale, _maxScaleFactor);
+        }
+    }
+}

+ 7 - 1
Ryujinx/Program.cs

@@ -18,12 +18,14 @@ namespace Ryujinx
 {
     class Program
     {
+        public static double WindowScaleFactor { get; private set; }
+
         public static string Version { get; private set; }
 
         public static string ConfigurationPath { get; set; }
 
         static void Main(string[] args)
-        {
+        { 
             // Parse Arguments.
             string launchPathArg      = null;
             string baseDirPathArg     = null;
@@ -54,6 +56,10 @@ namespace Ryujinx
                 }
             }
 
+            // Make process DPI aware for proper window sizing on high-res screens.
+            ForceDpiAware.Windows();
+            WindowScaleFactor = ForceDpiAware.GetWindowScaleFactor();
+
             // Delete backup files after updating.
             Task.Run(Updater.CleanupUpdate);
 

+ 8 - 2
Ryujinx/Ui/MainWindow.cs

@@ -1113,7 +1113,10 @@ namespace Ryujinx.Ui
 
         private void Settings_Pressed(object sender, EventArgs args)
         {
-            new SettingsWindow(this, _virtualFileSystem, _contentManager).Show();
+            SettingsWindow settingsWindow = new SettingsWindow(this, _virtualFileSystem, _contentManager);
+
+            settingsWindow.SetSizeRequest((int)(settingsWindow.DefaultWidth * Program.WindowScaleFactor), (int)(settingsWindow.DefaultHeight * Program.WindowScaleFactor));
+            settingsWindow.Show();
         }
 
         private void Simulate_WakeUp_Message_Pressed(object sender, EventArgs args)
@@ -1134,7 +1137,10 @@ namespace Ryujinx.Ui
 
         private void About_Pressed(object sender, EventArgs args)
         {
-            new AboutWindow().Show();
+            AboutWindow aboutWindow = new AboutWindow();
+
+            aboutWindow.SetSizeRequest((int)(aboutWindow.DefaultWidth * Program.WindowScaleFactor), (int)(aboutWindow.DefaultHeight * Program.WindowScaleFactor));
+            aboutWindow.Show();
         }
 
         private void Fav_Toggled(object sender, EventArgs args)

+ 4 - 1
Ryujinx/Ui/Windows/SettingsWindow.cs

@@ -570,7 +570,10 @@ namespace Ryujinx.Ui.Windows
         {
             ((ToggleButton)sender).SetStateFlags(StateFlags.Normal, true);
 
-            new ControllerWindow(playerIndex).Show();
+            ControllerWindow controllerWindow = new ControllerWindow(playerIndex);
+
+            controllerWindow.SetSizeRequest((int)(controllerWindow.DefaultWidth * Program.WindowScaleFactor), (int)(controllerWindow.DefaultHeight * Program.WindowScaleFactor));
+            controllerWindow.Show();
         }
 
         private void SaveToggle_Activated(object sender, EventArgs args)