[Settings]Fix ZoomIt page changing current directory (#37052)

microsoft · Jan 24, 2025 · fb35c93 · fb35c93
1 parent eeeabc8
commit fb35c93
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/.github/actions/spell-check/expect.txt b/.github/actions/spell-check/expect.txt
@@ -993,6 +993,7 @@ NNN
 NOACTIVATE
 NOAGGREGATION
 NOASYNC
+NOCHANGEDIR
 NOCLIP
 NOCLOSEPROCESS
 NOCOALESCE

diff --git a/src/settings-ui/Settings.UI/Helpers/OpenFileName.cs b/src/settings-ui/Settings.UI/Helpers/OpenFileName.cs
@@ -33,4 +33,10 @@ public class OpenFileName
         public IntPtr Hook = IntPtr.Zero;
         public string Template;
     }
+
+    // https://learn.microsoft.com/en-us/windows/win32/api/commdlg/ns-commdlg-openfilenamea
+    public enum OpenFileNameFlags
+    {
+        OFN_NOCHANGEDIR = 0x00000008,
+    }
 }
diff --git a/src/settings-ui/Settings.UI/SettingsXAML/Views/ZoomItPage.xaml.cs b/src/settings-ui/Settings.UI/SettingsXAML/Views/ZoomItPage.xaml.cs
@@ -35,6 +35,7 @@ private static string PickFileDialog(string filter, string title, string initial
             openFileName.Title = title;
             openFileName.FilterIndex = initialFilter;
             openFileName.DefExt = null;
+            openFileName.Flags = (int)OpenFileNameFlags.OFN_NOCHANGEDIR; // OFN_NOCHANGEDIR flag is needed, because otherwise GetOpenFileName overwrites the process working directory.
             IntPtr windowHandle = WinRT.Interop.WindowNative.GetWindowHandle(App.GetSettingsWindow());
             openFileName.Hwnd = windowHandle;