nits and such

Author: zadjii-msft

src/cascadia/TerminalApp/AppActionHandlers.cpp

@@ -1426,14 +1426,14 @@ namespace winrt::TerminalApp::implementation
     {
         if (Feature_ScratchpadPane::IsEnabled())
         {
-            auto scratchPane{ winrt::make_self<ScratchpadContent>() };
+            const auto& scratchPane{ winrt::make_self<ScratchpadContent>() };
 
             // This is maybe a little wacky - add our key event handler to the pane
             // we made. So that we can get actions for keys that the content didn't
             // handle.
             scratchPane->GetRoot().KeyDown({ this, &TerminalPage::_KeyDownHandler });
 
-            auto resultPane = std::make_shared<Pane>(*scratchPane);
+            const auto resultPane = std::make_shared<Pane>(*scratchPane);
             _SplitPane(_GetFocusedTabImpl(), SplitDirection::Automatic, 0.5f, resultPane);
             args.Handled(true);
         }

src/cascadia/TerminalApp/ScratchpadContent.cpp

@@ -15,8 +15,7 @@ namespace winrt::TerminalApp::implementation
     ScratchpadContent::ScratchpadContent()
     {
         _root = winrt::Windows::UI::Xaml::Controls::Grid{};
-        _root.VerticalAlignment(VerticalAlignment::Stretch);
-        _root.HorizontalAlignment(HorizontalAlignment::Stretch);
+        // Vertical and HorizontalAlignment are Stretch by default
 
         auto res = Windows::UI::Xaml::Application::Current().Resources();
         auto bg = res.Lookup(winrt::box_value(L"UnfocusedBorderBrush"));

src/features.xml

@@ -194,6 +194,7 @@
         <stage>AlwaysDisabled</stage>
         <alwaysEnabledBrandingTokens>
             <brandingToken>Dev</brandingToken>
+            <brandingToken>Canary</brandingToken>
         </alwaysEnabledBrandingTokens>
     </feature>