clean up more TODOs

Author: carlos-zamora

src/cascadia/TerminalSettingsEditor/MainPage.cpp

@@ -212,8 +212,8 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
             else if (const auto stringNavArg = navigationArg.try_as<hstring>())
             {
                 // The hstring navArg can be...
-                // - a color scheme name
-                // - an hstring tag (all defined at the top of the file)
+                // - color scheme name
+                // - hstring tag (all defined at the top of the file)
                 // Regardless, we'll use the font icon used by the navigation view item
                 WUX::Controls::FontIcon icon{};
                 icon.FontFamily(Media::FontFamily{ L"Segoe Fluent Icons, Segoe MDL2 Assets" });
@@ -1439,7 +1439,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
             filteredIndex.clear();
             for (const auto& entry : searchIndex)
             {
-                // TODO CARLOS: replace with fuzzy search
                 // Check for a match with DisplayText (i.e. "Globals_DefaultProfile/Header") and HelpText (i.e. "Globals_DefaultProfile/HelpText")
                 // in language neutral and current language
                 if (til::contains_linguistic_insensitive(entry.Entry->DisplayTextLocalized, queryText) ||

src/cascadia/TerminalSettingsEditor/Utils.h

@@ -137,8 +137,6 @@ struct HasScrollViewer
                     // or else the call to StartBringIntoView()
                     // will end up doing nothing
                     controlToFocus.StartBringIntoView();
-
-                    // TODO CARLOS: ensure this works in all scenarios (easiest to test when navigating to page by keyboard)
                     controlToFocus.Focus(winrt::Windows::UI::Xaml::FocusState::Programmatic);
                 }
                 page->_loadedRevoker.revoke();