Introducing Windows Customization by nieubank · Pull Request #2383 · microsoft/devhome

@nieubank marked this pull request as draft

March 11, 2024 16:55

@nieubank nieubank linked an issue

Mar 11, 2024

that may be closed by this pull request

krschau

@nieubank nieubank marked this pull request as ready for review

March 11, 2024 21:50

skanda890

skanda890

dhoehna

dhoehna

dhoehna

dhoehna

dhoehna

dhoehna

dhoehna

This commit includes the initial placeholder L1 page for 'Windows customization' which will host various settings and utilities to customize Windows to a developer's preferences.

The MainPage acts as a root which eventually will host a ContentPresenter for displaying the active view (for example, when search is added the ContentPresent will switch the root view temporarily to the 'All settings' search view, and once we get better support to integrate into the navigation service, sub-pages could be consolidated into that content presenter as well.

To provide an example of how options can be incorporated, the Developer File Explorer page was introduced.

The pattern we’re going for here is to provide logical settings groups in the form of a UserControl, which is then wrapped by a Page. This helps in the future when implementing search because we want to host all settings controls in the same root view and filter visibility based on the search filter. Having each group as a separate control view can easily enable hosting that UI in any form we need to down the line. The sub-pages themselves are probably temporarily placeholders for a better integration with the navigation service down the line.

See tools\Customization\DevHome.Customization\Customization.md for details.

@nieubank

nieubank

@nieubank

- Update to incorporate #2356 for PageService sub-page registration

- Update nuget package based on conflicts with newly added C++ projects that require packages.config

- Remove breadcrumb until we figure out if there's going to be a common way to do this without duplicating so much code. The impact here is minimal since Customization doesn't have a ton of sub-pages (yet)

- Mark SettingsChanged telemetry event to be a measure, I'm guessing this doesn't need to be Critical?

@nieubank

EricJohnson327

@nieubank nieubank deleted the feature/customization branch

May 8, 2024 21:55