Add support for getting DPI from the window (SDL)

Author: Ethereal77

sources/engine/Stride.Games/SDL/GameFormSDL.cs

@@ -15,7 +15,8 @@ namespace Stride.Games
     /// </summary>
     public class GameFormSDL : Window
     {
-#region Initialization
+        #region Initialization
+
         /// <summary>
         /// Initializes a new instance of the 'GameForm' class.
         /// </summary>
@@ -39,9 +40,11 @@ public GameFormSDL(string text) : base(text)
             RestoredActions += GameForm_RestoredActions;
             KeyDownActions += GameFormSDL_KeyDownActions;
         }
+
         #endregion
 
         #region Events
+
         /// <summary>
         /// Occurs when [app activated].
         /// </summary>
@@ -75,6 +78,7 @@ public GameFormSDL(string text) : base(text)
         #endregion
 
         #region Implementation
+
         // TODO: The code below is taken from GameForm.cs of the Windows Desktop implementation. This needs reviewing
         private FormWindowState previousWindowState;
 

sources/engine/Stride.Games/SDL/GameWindowSDL.cs

@@ -109,7 +109,7 @@ protected override void Initialize(GameContext<Window> gameContext)
 
             window.ClientSize = new Size2(width, height);
 
-            window.MouseEnterActions += WindowOnMouseEnterActions;   
+            window.MouseEnterActions += WindowOnMouseEnterActions;
             window.MouseLeaveActions += WindowOnMouseLeaveActions;
 
             var gameForm = window as GameFormSDL;
@@ -120,7 +120,7 @@ protected override void Initialize(GameContext<Window> gameContext)
                 gameForm.SizeChanged += OnClientSizeChanged;
                 gameForm.CloseActions += GameForm_CloseActions;
                 gameForm.FullscreenToggle += OnFullscreenToggle;
-                
+                gameForm.DisplayChangedActions += WindowOnDisplayChangedActions;
             }
             else
             {
@@ -380,6 +380,12 @@ public override bool Focused
             }
         }
 
+        private void WindowOnDisplayChangedActions(WindowEvent sdlWindowEvent)
+        {
+            Dpi = new Int2((int) window.Dpi);
+            DpiScale = window.DpiScale;
+        }
+
         protected override void Destroy()
         {
             if (window != null)

sources/engine/Stride.Graphics/SDL/Window.cs

@@ -15,23 +15,23 @@ public unsafe class Window : IDisposable
 
         private Silk.NET.SDL.Window* sdlHandle;
 
-#region Initialization
+        #region Initialization
 
         /// <summary>
         /// Initializes static members of the <see cref="Window"/> class.
         /// </summary>
         static Window()
         {
-            SDL = Silk.NET.SDL.Sdl.GetApi();
+            SDL = Sdl.GetApi();
 
-            // jklawreszuk: Workaround for wayland (see #2487 for more details)  
+            // jklawreszuk: Workaround for wayland (see #2487 for more details)
             // TODO: Wayland SDL_EGL_MakeCurrent does not cover multi-context scenario (https://github.com/libsdl-org/SDL/issues/9072)
             if (OperatingSystem.IsLinux())
                 SDL.SetHint("SDL_VIDEODRIVER", "x11");
 
             SDL.Init(Sdl.InitEverything);
 
-            // Pass first mouse event when user clicked on window 
+            // Pass first mouse event when user clicked on window
             SDL.SetHint(Sdl.HintMouseFocusClickthrough, "1");
 
             // Don't leave fullscreen on focus loss
@@ -73,11 +73,10 @@ public Window(string title, IntPtr parent)
                 // Create the SDL window and then extract the native handle.
                 sdlHandle = SDL.CreateWindow(title, Sdl.WindowposUndefined, Sdl.WindowposUndefined, 640, 480, (uint)flags);
             }
-            
 
             if (sdlHandle == null)
             {
-                throw new Exception("Cannot allocate SDL Window: " + SDL.GetErrorS()); 
+                throw new Exception("Cannot allocate SDL Window: " + SDL.GetErrorS());
             }
 
             SysWMInfo info = default;
@@ -105,6 +104,13 @@ public Window(string title, IntPtr parent)
             {
                 Handle = (IntPtr)info.Info.Cocoa.Window;
             }
+
+            var displayIndex = SDL.GetWindowDisplayIndex(sdlHandle);
+            if (displayIndex == -1)
+                displayIndex = 0;
+
+            DisplayIndex = displayIndex;
+
             Application.RegisterWindow(this);
             Application.ProcessEvents();
         }
@@ -321,7 +327,7 @@ public Size2 Size
             }
             set { SDL.SetWindowSize(sdlHandle, value.Width, value.Height); }
         }
-        
+
         /// <summary>
         /// The opacity of the window.
         /// </summary>
@@ -446,6 +452,30 @@ public FormBorderStyle FormBorderStyle
             }
         }
 
+        /// <summary>
+        /// DPI (dots per inch) of the Window.
+        /// </summary>
+        public float Dpi
+        {
+            get
+            {
+                float ddpi, hdpi, vdpi;
+
+                if (SDL.GetDisplayDPI(DisplayIndex, &ddpi, &hdpi, &vdpi) != 0)
+                {
+                    // Failed to get DPI, return a default value of 96 which is the standard DPI for many platforms.
+                    return 96.0f;
+                }
+
+                return ddpi;
+            }
+        }
+
+        /// <summary>
+        /// DPI scaling factor of the Window. 100 % corresponds to a DPI of 96.
+        /// </summary>
+        public float DpiScale => Dpi / 96.0f;
+
         public void SetRelativeMouseMode(bool enabled)
         {
             SDL.SetRelativeMouseMode(enabled ? SdlBool.True : SdlBool.False);
@@ -489,6 +519,7 @@ public void SetRelativeMouseMode(bool enabled)
         public event WindowEventDelegate MouseLeaveActions;
         public event WindowEventDelegate FocusGainedActions;
         public event WindowEventDelegate FocusLostActions;
+        public event WindowEventDelegate DisplayChangedActions;
         public event DropEventDelegate DropFileActions;
 
         /// <summary>
@@ -554,7 +585,7 @@ public virtual void ProcessEvent(Event e)
                 case EventType.Fingerup:
                     FingerReleaseActions?.Invoke(e.Tfinger);
                     break;
-                
+
                 case EventType.Dropfile:
                     DropFileActions?.Invoke(Silk.NET.Core.Native.SilkMarshal.PtrToString((IntPtr)e.Drop.File, Silk.NET.Core.Native.NativeStringEncoding.UTF8));
                     break;
@@ -610,12 +641,27 @@ public virtual void ProcessEvent(Event e)
                         case WindowEventID.FocusLost:
                             FocusLostActions?.Invoke(e.Window);
                             break;
-                    }
+
+                        case WindowEventID.DisplayChanged:
+                            OnDisplayChanged(e.Window);
+                            break;
+                        }
                     break;
                 }
             }
         }
 
+        private void OnDisplayChanged(WindowEvent e)
+        {
+            var displayIndex = SDL.GetWindowDisplayIndex(sdlHandle);
+            if (displayIndex == -1)
+                displayIndex = 0;
+
+            DisplayIndex = displayIndex;
+
+            DisplayChangedActions?.Invoke(e);
+        }
+
         /// <summary>
         /// Platform specific handle for Window.
         /// </summary>
@@ -632,6 +678,11 @@ public virtual void ProcessEvent(Event e)
         /// </summary>
         public IntPtr Display { get; private set; }
 
+        /// <summary>
+        /// Index of the display where the current Window is being shown.
+        /// </summary>
+        public int DisplayIndex { get; private set; }
+
         /// <summary>
         /// Surface of current Window (valid only for Android).
         /// </summary>
@@ -650,7 +701,8 @@ public bool Exists
             get { return SdlHandle != IntPtr.Zero; }
         }
 
-#region Disposal
+        #region Disposal
+
         ~Window()
         {
             Dispose(false);
@@ -690,7 +742,7 @@ protected virtual void Dispose(bool disposing)
                 Handle = IntPtr.Zero;
             }
         }
-  
+
         // This code added to correctly implement the disposable pattern.
         public void Dispose()
         {