Address review: handle NavigationFailed gracefully without rethrowing

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Fix NullReferenceException in Frame_NavigationFailed when e.Exception is null
2026-07-02 08:28:55 +02:00 · 2026-05-14 16:20:42 +08:00 · 2026-04-29 08:49:32 +00:00 · 2026-04-29 08:48:31 +00:00
2 changed files with 29 additions and 2 deletions
--- a/README.md
+++ b/README.md
@@ -47,7 +47,22 @@ But to get started quickly, choose one of the installation methods below:
 <summary><strong>Download the .exe file from GitHub</strong></summary>
 <br/>

-Go to the [PowerToys GitHub releases](https://aka.ms/installPowerToys) page, scroll down and select **Assets** to reveal the installation files, and choose the one that matches your architecture and install scope. For most devices, that would be _x64 per-user_.
+Go to the [PowerToys GitHub releases](https://aka.ms/installPowerToys), select **Assets** to reveal the installation files, and choose the one that matches your architecture and install scope. For most devices, that would be _x64 per-user_.
+
+<!-- items that need to be updated release to release -->
+[github-next-release-work]: https://github.com/microsoft/PowerToys/issues?q=is%3Aissue+milestone%3A%22PowerToys+0.100%22
+[ptUserX64]: https://github.com/microsoft/PowerToys/releases/download/v0.99.0/PowerToysUserSetup-0.99.0-x64.exe
+[ptUserArm64]: https://github.com/microsoft/PowerToys/releases/download/v0.99.0/PowerToysUserSetup-0.99.0-arm64.exe
+[ptMachineX64]: https://github.com/microsoft/PowerToys/releases/download/v0.99.0/PowerToysSetup-0.99.0-x64.exe
+[ptMachineArm64]: https://github.com/microsoft/PowerToys/releases/download/v0.99.0/PowerToysSetup-0.99.0-arm64.exe
+
+| Description | Filename |
+| --- | --- |
+| Per user - x64       | [PowerToysUserSetup-0.99.0-x64.exe][ptUserX64] |
+| Per user - ARM64     | [PowerToysUserSetup-0.99.0-arm64.exe][ptUserArm64] |
+| Machine wide - x64   | [PowerToysSetup-0.99.0-x64.exe][ptMachineX64] |
+| Machine wide - ARM64 | [PowerToysSetup-0.99.0-arm64.exe][ptMachineArm64] |
+
 </details>

 <details>
--- a/src/settings-ui/Settings.UI/ViewModels/ShellViewModel.cs
+++ b/src/settings-ui/Settings.UI/ViewModels/ShellViewModel.cs
@@ -9,6 +9,7 @@ using System.Linq;
 using System.Threading.Tasks;
 using System.Windows.Input;

+using ManagedCommon;
 using Microsoft.PowerToys.Settings.UI.Helpers;
 using Microsoft.PowerToys.Settings.UI.Library;
 using Microsoft.PowerToys.Settings.UI.Library.Helpers;
@@ -135,7 +136,18 @@ namespace Microsoft.PowerToys.Settings.UI.ViewModels

        private void Frame_NavigationFailed(object sender, NavigationFailedEventArgs e)
        {
-            throw e.Exception;
+            var sourcePage = e.SourcePageType?.FullName ?? "<unknown>";
+
+            if (e.Exception is null)
+            {
+                Logger.LogWarning($"Navigation to '{sourcePage}' failed without an exception.");
+            }
+            else
+            {
+                Logger.LogError($"Navigation to '{sourcePage}' failed.", e.Exception);
+            }
+
+            e.Handled = true;
        }

        private void Frame_Navigated(object sender, NavigationEventArgs e)
Author	SHA1	Message	Date
Muyuan Li (from Dev Box)	f9679b937d	Address review: handle NavigationFailed gracefully without rethrowing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>	2026-05-14 16:20:42 +08:00
copilot-swe-agent[bot]	36300d3c75	Fix NullReferenceException in Frame_NavigationFailed when e.Exception is null Agent-Logs-Url: https://github.com/microsoft/PowerToys/sessions/af982aa1-504b-47f1-9ec0-93b29602b2af Co-authored-by: MuyuanMS <116717757+MuyuanMS@users.noreply.github.com>	2026-04-29 08:49:32 +00:00
copilot-swe-agent[bot]	1cde68ae04	Initial plan	2026-04-29 08:48:31 +00:00