From 92833757c6515cb1024dc780199053e39389f7c1 Mon Sep 17 00:00:00 2001
From: Carlos Zamora <cazamor@microsoft.com>
Date: Tue, 12 Jan 2021 17:51:05 -0800
Subject: [PATCH] Prevent navigation to already selected menu item

---
 src/cascadia/TerminalSettingsEditor/MainPage.cpp | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/cascadia/TerminalSettingsEditor/MainPage.cpp b/src/cascadia/TerminalSettingsEditor/MainPage.cpp
index 4d51f517d97..3343bf163a8 100644
--- a/src/cascadia/TerminalSettingsEditor/MainPage.cpp
+++ b/src/cascadia/TerminalSettingsEditor/MainPage.cpp
@@ -206,6 +206,13 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
     {
         if (const auto clickedItemContainer = args.InvokedItemContainer())
         {
+            if (clickedItemContainer.IsSelected())
+            {
+                // Clicked on the selected item.
+                // Don't navigate to the same page again.
+                return;
+            }
+
             if (const auto navString = clickedItemContainer.Tag().try_as<hstring>())
             {
                 if (navString == addProfileTag)