Repository: microsoft/terminal (language: cpp)

Issue:
1.17 still displays launch params (Default,Default) despite values being set
<img width="510" alt="image" src="https://user-images.githubusercontent.com/189190/204926251-b3663c5e-694b-4b0d-9cdb-649d9487f49f.png">


Discussion / hints:

Fix patch:
diff --git a/src/cascadia/TerminalSettingsEditor/LaunchViewModel.cpp b/src/cascadia/TerminalSettingsEditor/LaunchViewModel.cpp
index bd889eae2c8..2ef854033c1 100644
--- a/src/cascadia/TerminalSettingsEditor/LaunchViewModel.cpp
+++ b/src/cascadia/TerminalSettingsEditor/LaunchViewModel.cpp
@@ -68,7 +68,7 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
         const auto x = _Settings.GlobalSettings().InitialPosition().X;
         // If there's no value here, return NAN - XAML will ignore it and
         // put the placeholder text in the box instead
-        const auto xCoord = x.try_as<int64_t>();
+        const auto xCoord = x.try_as<int32_t>();
         return xCoord.has_value() ? gsl::narrow_cast<double>(xCoord.value()) : NAN;
     }
 
@@ -77,7 +77,7 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
         const auto y = _Settings.GlobalSettings().InitialPosition().Y;
         // If there's no value here, return NAN - XAML will ignore it and
         // put the placeholder text in the box instead
-        const auto yCoord = y.try_as<int64_t>();
+        const auto yCoord = y.try_as<int32_t>();
         return yCoord.has_value() ? gsl::narrow_cast<double>(yCoord.value()) : NAN;
     }
 


Test patch: