When we add a new tab in compact mode, re-enforce Compact mode (#6670)

This workaround was suggested by @chingucoding in
microsoft/microsoft-ui-xaml#2711

Fixes #6570

## References

microsoft/microsoft-ui-xaml#2711
#6570

## PR Checklist
* [x] Closes an issue
* [x] CLA
* [x] Tested
* [x] Docs not required
* [x] Schema not required

(cherry picked from commit fefd1408f2)

res/README.md

@@ -17,5 +17,5 @@ Please consult the [license](https://raw.githubusercontent.com/microsoft/cascadi
 
 ### Fonts Included
 
-* Cascadia Code, Cascadia Mono (2005.15)
-   * from microsoft/cascadia-code@0610f2df4356200adb93cb5bca2221b92ad6ee7e
+* Cascadia Code, Cascadia Mono (2007.01)
+   * from microsoft/cascadia-code@311cc603f30635da704b6a7d13050e245e61667b

src/cascadia/TerminalApp/TerminalPage.cpp

@@ -557,6 +557,16 @@ namespace winrt::TerminalApp::implementation
 
         auto tabViewItem = newTabImpl->GetTabViewItem();
         _tabView.TabItems().Append(tabViewItem);
+        // GH#6570
+        // The TabView does not apply compact sizing to items added after Compact is enabled.
+        // By forcibly reapplying compact sizing every time we add a new tab, we'll make sure
+        // that it works.
+        // Workaround from https://github.com/microsoft/microsoft-ui-xaml/issues/2711
+        if (_tabView.TabWidthMode() == MUX::Controls::TabViewWidthMode::Compact)
+        {
+            _tabView.UpdateLayout();
+            _tabView.TabWidthMode(MUX::Controls::TabViewWidthMode::Compact);
+        }
 
         // Set this tab's icon to the icon from the user's profile
         const auto* const profile = _settings->FindProfile(profileGuid);

src/cascadia/TerminalApp/defaults.json

@@ -30,7 +30,7 @@
         {
             "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
             "name": "Windows PowerShell",
-            "commandline": "powershell.exe",
+            "commandline": "%SystemRoot%\\System32\\WindowsPowerShell\\v1.0\\powershell.exe",
             "icon": "ms-appx:///ProfileIcons/{61c54bbd-c2c6-5271-96e7-009a87ff44bf}.png",
             "colorScheme": "Campbell",
             "antialiasingMode": "grayscale",
@@ -49,7 +49,7 @@
         {
             "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
             "name": "Command Prompt",
-            "commandline": "cmd.exe",
+            "commandline": "%SystemRoot%\\System32\\cmd.exe",
             "icon": "ms-appx:///ProfileIcons/{0caa0dad-35be-5f56-a8ff-afceeeaa6101}.png",
             "colorScheme": "Campbell",
             "antialiasingMode": "grayscale",

src/renderer/dx/CustomTextLayout.cpp

@@ -41,8 +41,7 @@ CustomTextLayout::CustomTextLayout(gsl::not_null<IDWriteFactory1*> const factory
     _runs{},
     _breakpoints{},
     _runIndex{ 0 },
-    _width{ width },
-    _isEntireTextSimple{ false }
+    _width{ width }
 {
     // Fetch the locale name out once now from the format
     _localeName.resize(gsl::narrow_cast<size_t>(format->GetLocaleNameLength()) + 1); // +1 for null
@@ -77,7 +76,6 @@ CustomTextLayout::CustomTextLayout(gsl::not_null<IDWriteFactory1*> const factory
     RETURN_HR_IF_NULL(E_INVALIDARG, columns);
     *columns = 0;
 
-    RETURN_IF_FAILED(_AnalyzeTextComplexity());
     RETURN_IF_FAILED(_AnalyzeRuns());
     RETURN_IF_FAILED(_ShapeGlyphRuns());
 
@@ -108,7 +106,6 @@ CustomTextLayout::CustomTextLayout(gsl::not_null<IDWriteFactory1*> const factory
                                                                FLOAT originX,
                                                                FLOAT originY) noexcept
 {
-    RETURN_IF_FAILED(_AnalyzeTextComplexity());
     RETURN_IF_FAILED(_AnalyzeRuns());
     RETURN_IF_FAILED(_ShapeGlyphRuns());
     RETURN_IF_FAILED(_CorrectGlyphRuns());
@@ -122,44 +119,6 @@ CustomTextLayout::CustomTextLayout(gsl::not_null<IDWriteFactory1*> const factory
     return S_OK;
 }
 
-// Routine Description:
-// - Uses the internal text information and the analyzers/font information from construction
-//   to determine the complexity of the text. If the text is determined to be entirely simple,
-//   we'll have more chances to optimize the layout process.
-// Arguments:
-// - <none> - Uses internal state
-// Return Value:
-// - S_OK or suitable DirectWrite or STL error code
-[[nodiscard]] HRESULT CustomTextLayout::_AnalyzeTextComplexity() noexcept
-{
-    try
-    {
-        const auto textLength = gsl::narrow<UINT32>(_text.size());
-
-        BOOL isTextSimple = FALSE;
-        UINT32 uiLengthRead = 0;
-
-        // Start from the beginning.
-        const UINT32 glyphStart = 0;
-
-        _glyphIndices.resize(textLength);
-
-        const HRESULT hr = _analyzer->GetTextComplexity(
-            _text.c_str(),
-            textLength,
-            _font.Get(),
-            &isTextSimple,
-            &uiLengthRead,
-            &_glyphIndices.at(glyphStart));
-
-        RETURN_IF_FAILED(hr);
-
-        _isEntireTextSimple = isTextSimple && uiLengthRead == textLength;
-    }
-    CATCH_RETURN();
-    return S_OK;
-}
-
 // Routine Description:
 // - Uses the internal text information and the analyzers/font information from construction
 //   to determine the complexity of the text inside this layout, compute the subsections (or runs)
@@ -190,7 +149,11 @@ CustomTextLayout::CustomTextLayout(gsl::not_null<IDWriteFactory1*> const factory
         // Allocate enough room to have one breakpoint per code unit.
         _breakpoints.resize(_text.size());
 
-        if (!_isEntireTextSimple)
+        BOOL isTextSimple = FALSE;
+        UINT32 uiLengthRead = 0;
+        RETURN_IF_FAILED(_analyzer->GetTextComplexity(_text.c_str(), textLength, _font.Get(), &isTextSimple, &uiLengthRead, NULL));
+
+        if (!(isTextSimple && uiLengthRead == _text.size()))
         {
             // Call each of the analyzers in sequence, recording their results.
             RETURN_IF_FAILED(_analyzer->AnalyzeLineBreakpoints(this, 0, textLength, this));
@@ -311,39 +274,6 @@ CustomTextLayout::CustomTextLayout(gsl::not_null<IDWriteFactory1*> const factory
             _glyphIndices.resize(totalGlyphsArrayCount);
         }
 
-        if (_isEntireTextSimple)
-        {
-            // When the entire text is simple, we can skip GetGlyphs and directly retrieve glyph indices and
-            // advances(in font design unit). With the help of font metrics, we can calculate the actual glyph
-            // advances without the need of GetGlyphPlacements. This shortcut will significantly reduce the time
-            // needed for text analysis.
-            DWRITE_FONT_METRICS1 metrics;
-            run.fontFace->GetMetrics(&metrics);
-
-            // With simple text, there's only one run. The actual glyph count is the same as textLength.
-            _glyphDesignUnitAdvances.resize(textLength);
-            _glyphAdvances.resize(textLength);
-            _glyphOffsets.resize(textLength);
-
-            USHORT designUnitsPerEm = metrics.designUnitsPerEm;
-
-            RETURN_IF_FAILED(_font->GetDesignGlyphAdvances(
-                textLength,
-                &_glyphIndices.at(glyphStart),
-                &_glyphDesignUnitAdvances.at(glyphStart),
-                run.isSideways));
-
-            for (size_t i = glyphStart; i < _glyphAdvances.size(); i++)
-            {
-                _glyphAdvances.at(i) = (float)_glyphDesignUnitAdvances.at(i) / designUnitsPerEm * _format->GetFontSize() * run.fontScale;
-            }
-
-            run.glyphCount = textLength;
-            glyphStart += textLength;
-
-            return S_OK;
-        }
-
         std::vector<DWRITE_SHAPING_TEXT_PROPERTIES> textProps(textLength);
         std::vector<DWRITE_SHAPING_GLYPH_PROPERTIES> glyphProps(maxGlyphCount);
 
@@ -441,12 +371,6 @@ CustomTextLayout::CustomTextLayout(gsl::not_null<IDWriteFactory1*> const factory
 {
     try
     {
-        // For simple text, there is no need to correct runs.
-        if (_isEntireTextSimple)
-        {
-            return S_OK;
-        }
-
         // Correct each run separately. This is needed whenever script, locale,
         // or reading direction changes.
         for (UINT32 runIndex = 0; runIndex < _runs.size(); ++runIndex)
@@ -584,6 +508,9 @@ try
 
     // We're going to walk through and check for advances that don't match the space that we expect to give out.
 
+    DWRITE_FONT_METRICS1 metrics;
+    run.fontFace->GetMetrics(&metrics);
+
     // Glyph Indices represents the number inside the selected font where the glyph image/paths are found.
     // Text represents the original text we gave in.
     // Glyph Clusters represents the map between Text and Glyph Indices.

src/renderer/dx/CustomTextLayout.h

@@ -134,7 +134,6 @@ namespace Microsoft::Console::Render
         [[nodiscard]] HRESULT STDMETHODCALLTYPE _AnalyzeBoxDrawing(gsl::not_null<IDWriteTextAnalysisSource*> const source, UINT32 textPosition, UINT32 textLength);
         [[nodiscard]] HRESULT STDMETHODCALLTYPE _SetBoxEffect(UINT32 textPosition, UINT32 textLength);
 
-        [[nodiscard]] HRESULT _AnalyzeTextComplexity() noexcept;
         [[nodiscard]] HRESULT _AnalyzeRuns() noexcept;
         [[nodiscard]] HRESULT _ShapeGlyphRuns() noexcept;
         [[nodiscard]] HRESULT _ShapeGlyphRun(const UINT32 runIndex, UINT32& glyphStart) noexcept;
@@ -181,9 +180,6 @@ namespace Microsoft::Console::Render
 
         // Glyph shaping results
 
-        // Whether the entire text is determined to be simple and does not require full script shaping.
-        bool _isEntireTextSimple;
-
         std::vector<DWRITE_GLYPH_OFFSET> _glyphOffsets;
 
         // Clusters are complicated. They're in respect to each individual run.
@@ -195,9 +191,6 @@ namespace Microsoft::Console::Render
         // This appears to be the index of the glyph inside each font.
         std::vector<UINT16> _glyphIndices;
 
-        // This is for calculating glyph advances when the entire text is simple.
-        std::vector<INT32> _glyphDesignUnitAdvances;
-
         std::vector<float> _glyphAdvances;
 
         struct ScaleCorrection