Fragments should be able to add actions #20583

Closed
opened 2026-01-31 07:18:21 +00:00 by claunia · 1 comment
claunia added the Issue-FeatureArea-SettingsIn-PRNeeds-Tag-FixProduct-Terminal labels 2026-01-31 07:18:21 +00:00
Author
Owner

@zadjii-msft commented on GitHub (Sep 29, 2023):

I think we'd need to:

  • Add an internal / private source to actions, to indicate where we got the action from (ala profiles)
  • in SettingsLoader::_parseFragment move them all over into the user settings
  • don't serialize ones back that weren't from the user settings

NOPE not that

  • GlobalAppSettings::_FinalizeInheritance sets the LeastImportantParent for the user's action map on each parent global settings. Well, easy enough.
  • Create a "blank" GAS for each fragment
  • don't parse keys
  • add as a parent of the user's GAS
@zadjii-msft commented on GitHub (Sep 29, 2023): I think we'd need to: * ~Add an internal / private `source` to actions, to indicate where we got the action from (ala profiles)~ * ~in `SettingsLoader::_parseFragment` move them all over into the user settings~ * ~don't serialize ones back that weren't from the user settings~ NOPE not that * `GlobalAppSettings::_FinalizeInheritance` sets the LeastImportantParent for the user's action map on _each_ parent global settings. Well, easy enough. * Create a "blank" GAS for each fragment * don't parse keys * add as a parent of the user's GAS
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#20583