Port SettingsCard/Expander from WCT and update SUI to use it#20232
Port SettingsCard/Expander from WCT and update SUI to use it#20232carlos-zamora wants to merge 16 commits into
Conversation
This comment has been minimized.
This comment has been minimized.
|
UI changes I noticed while poking around:
I hit a point where I could go in and make everything look the same as we have it now, but I felt that that mildly contradicted the idea behind trying to make the UI "consistent" by using these controls. @niels9001 I'll go through and do a second pass fixing the things above. Can you also go through it and point out any inconsistencies with the UI that should be addressed? The main files that are probably of interest are:
Once we figure that out and this gets reviewed/merged, we can probably use |
The issue is that we are now mixing-and-matching How big is the gap between |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
834a37e to
f7eb2fb
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Aside from that, this PR is ready for review. Unfortunately, it's a decently large thing to figure out 😅 |
Summary of the Pull Request
Ports the
SettingsCardandSettingsExpanderfrom Windows Community Toolkit to the TerminalSettingsEditor project:SettingsCardandSettingsExpanderwere ported over and reuse our macros to reduce boilerplate.Descriptionis applied asAutomationProperties.FullDescriptionfor 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 asHelpText. This change exposes it asFullDescriptionwhich 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.SettingsControlsHelpersis 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
SettingContainerentirely. Associated changes:.HelpText-->.DescriptionSettingContainerStyle.xamland moving anything that's still used toCommonResources.xamlBackgroundImageSettingsVisible-->BackgroundImageSettingsEnabledto make use of the disabled stylingOther notable changes as a part of using the new controls:
SettingsCardSwitching 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
PR Checklist
Related to #17000