Skip to content

Update profile subpages#20336

Open
carlos-zamora wants to merge 3 commits into
dev/cazamor/sui-rejuv/setting-containerfrom
dev/cazamor/sui-rejuv/profile-rejuv
Open

Update profile subpages#20336
carlos-zamora wants to merge 3 commits into
dev/cazamor/sui-rejuv/setting-containerfrom
dev/cazamor/sui-rejuv/profile-rejuv

Conversation

@carlos-zamora

Copy link
Copy Markdown
Member

Summary of the Pull Request

Updates the individual profile pages to include:

  • icons + settings expanders/cards
  • a new "reset value" button
  • a new "unfocused appearance" subpage with its own preview (similar to default appearance)

Detailed Description of the Pull Request / Additional comments

  • IInheritableViewModel: interface exposing HasSetting(), ClearSetting(), and SettingOverrideSource()
  • Profiles_UnfocusedAppearance.h/cpp/idl/xaml: the new unfocused appearance page

Validation Steps Performed

  • settings search + deep link navigation
  • "reset value" button

PR Checklist

Related to #17000

@carlos-zamora

Copy link
Copy Markdown
Member Author

Demo

Profiles_Base

image

Profiles_UnfocusedAppearance

image

Tooltips

image image

Comment on lines +397 to +402

// Optional resource name for a secondary label shown beneath the primary label in
// search results (i.e. "Nav_ProfileDefaults/Content" for the profile-defaults page links).
// Empty for most entries.
// NOTE: wrapped in USES_RESOURCE() (when set) for the same compile-time validation as ResourceName.
wil::zwstring_view SecondaryResourceName;

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The SecondaryResourceName changes in this file conflict with #20199 . They both have basically the same implementation with a different variable name. The consumer in SearchIndex.cpp also needs to be updated.

I'll go in and make this match the one from the other one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant