Carlos Zamora

Port SettingsCard/Expander from WCT and update SUI to use it (#20232) ## Summary of the Pull Request Ports the `SettingsCard` and `SettingsExpander` from Windows Community Toolkit to the TerminalSettingsEditor project: - Windows Community Toolkit code: https://github.com/CommunityToolkit/Windows/tree/main/components/SettingsControls/src - `SettingsCard` and `SettingsExpander` were ported over and reuse our macros to reduce boilerplate. - A few deviations exist since WCT targets WinUI 3 but TSE uses WinUI 2 (i.e. chevron rendering) - One deviation specific to TSE: `Description` is applied as `AutomationProperties.FullDescription` for accessibility. This is for accessibility compliance. Description text should be read out by the screen reader (scan mode isn't enough). Prior to this change, we would expose it as `HelpText`. This change exposes it as `FullDescription` which is more fitting as it's generally an opt-in behavior for screen reader users to hear. This prevents the user from being overwhelmed with information. - `SettingsControlsHelpers` is used to make this much easier to review. Any converters, triggers, or other classes that are used for the migration were added there. Additionally, this PR performs the migration to use these new controls and removes `SettingContainer` entirely. Associated changes: - Resources: renamed keys `.HelpText` --> `.Description` - Deleting `SettingContainerStyle.xaml` and moving anything that's still used to `CommonResources.xaml` - Appearances: `BackgroundImageSettingsVisible` --> `BackgroundImageSettingsEnabled` to make use of the disabled styling - GenerateSettingsIndex.ps1 was updated to target SettingsCard/SettingsExpander for the search index Other notable changes as a part of using the new controls: - Navigators (i.e. "Advanced" button in Profiles page, Extensions page) were also converted to use `SettingsCard` - Utils.h/cpp includes a fix for deep link to work for nested settings (i.e. "Foreground", "Columns") Switching to use the WCT port directly allows us to iterate faster on the settings UI and be more consistent with the Windows ecosystem. However, this does mean that the "reset button" had to be removed from the individual settings. This was already a weird pattern, so removing it shouldn't be too much of an issue. However, there was value to being able to reset settings. To provide this kind of functionality, a "reset" button is being added to the profile pages. This clears all settings in a profile so that they can apply their own customizations on top. It sacrifices granularity for simplicity and is a better understood concept. ## Validation Steps Performed - [X] settings search should pick up and navigate to settings properly - [X] accessibility audit ## PR Checklist Related to #17000

Update styling of Actions page (#20215) ## Summary of the Pull Request Updates the Actions and Edit Action pages according to the mockups in #20036 + some iterations past that. Notable changes: - "Actions" --> "Shortcuts" - `KeyChordVisual`: new control to display the key bindings. Bound to a `KeyChord`. Inspired by PowerToys' implementation - Actions.xaml: - relatively thin entries for each shortcut - Actions can have multiple key chords associated with them. All of these are shown. - Edit button associated with each key chord. Clicking it allows you to immediately modify the key chord without navigating to the sub-page. Tooltip + automation properties constructed to make it more clear what these buttons do. - EditAction.xaml: - each action arg is represented in a settings card for consistent UI - shortcut type is the first/primary setting. Changing this updates the "additional customizations" section at the bottom - "key bindings" section enumerates all associated shortcuts and allows the user to modify them individually - "additional customizations" includes "shortcut name" to allow users to override the command palette representation. This also includes action args UI. ## Validation Steps Performed Scenarios: - Actions page: - [X] key chord visuals are legible - [X] edit button --> edit mode --> show key chord listener (and focus it) - [X] binding to an key chord that is in use shows a warning - [X] clicking on entry navigates to Edit Action page - [X] accessibility testing - Edit Action page: - [X] accessibility testing ## PR Checklist Related to #17000

Introduce Profiles page to SUI (#20199) ## Summary of the Pull Request Updates the settings UI to introduce a new Profiles page. This page includes: - a "Profile Defaults" navigator - a "Color Schemes" navigator (removed top-level item in favor of this) - an "add profile" split button (removed AddProfile page in favor of this) - a navigator for each profile - tooltips were added for the "hidden"/"orphaned" info badge ## Validation Steps Performed Scenarios: - [x] settings search - [X] defaults - [X] color schemes - [X] setting in defaults - [X] breadcrumbs - [X] navigate to individual profiles (via search and not) - [X] update settings for a profile - [X] accessibility testing ## PR Checklist Related to #17000 Stacked on #20232

Refresh hovered hyperlink on scroll (#20357) ## Summary of the Pull Request Updates the cached hovered hyperlink whenever a scroll occurs. All changes were made in `ControlCore`: - Add `_refreshHoveredCell()` which simply sets the hovered cell again - Calls the new function in `UserScrollViewport()` for user-initiated scrolling and `updateScrollBar` for non-user-initiated scrolling ## Validation Steps Performed With multiple hyperlinks in the text buffer, the following scenarios resulted in the hovered hyperlink being updated properly: ✅ mouse wheel scrolling ✅ keyboard scrolling ✅ scroll to mark ## PR Checklist Closes #20219 Based on #20239

Fix Export Tab leaking Enter via Command Palette (#20358) ## Summary of the Pull Request Fixes a bug where Enter would leak into the terminal session when running the export buffer command from the command palette. Turns out this is because `_ExportTab()` opens the file picker synchronously in the key handler. The file picker spins a nested Win32 modal message loop, which pumps the queued `WM_CHAR` into the terminal. Switching to the foreground thread gets out of the nested modal loop, resulting in the `WM_CHAR` being handled through XAML's pipeline and suppressing it normally. The low priority makes it so that the command palette is visibly dismissed before opening the file picker. ## Validation Steps Performed Run export buffer action from the command palette. ✅ Enter does not go through to the underlying terminal. ## PR Checklist Closes #20188 Based on #20221

Fix action name inconsistently updating in Edit Action page (#20455) In the Edit Action page, the "action name" box wouldn't update sometimes when the "action type" was updated. `DisplayName()` would be refreshed, but the new name wouldn't be pulled from the model because the cache was still populated. This just invalidates it so that it's generated again. Same as PR #20381 (1.25 port)

Remove em-dash usage (#20611) Removes most of the em-dashes in our repo. Any related to the Settings UI are covered in #20336.