Add the ability to mark a profile as "read only" #16606

Open
opened 2026-01-31 05:17:05 +00:00 by claunia · 0 comments
Owner

Originally created by @zadjii-msft on GitHub (Feb 3, 2022).

From a mail thread.

Say an application author wants to add a fragment profile for their app. They want to turn on a bunch of specific settings for that app, like, closeOnExit. But they REALLY REALLY don't want users overriding those properties. The app author should be able to specify some way that indicates that this profile can no longer be layered on top of.

addenda:

  • when we have arbitrary profile layering (#2933 is similar but not what I was thinking), then does readOnly prevent a user profile from being layered by other profiles?
  • Do we throw a warning when the user tries to add properties in their settings to a readonly profile? (probably)
  • What happens if they mark profiles.defaults as readonly? (probably warn them, because that's an obvious footgun)
Originally created by @zadjii-msft on GitHub (Feb 3, 2022). From a mail thread. Say an application author wants to add a fragment profile for their app. They want to turn on a bunch of specific settings for that app, like, `closeOnExit`. But they REALLY REALLY don't want users overriding those properties. The app author should be able to specify some way that indicates that this profile can no longer be layered on top of. addenda: * when we have arbitrary profile layering (#2933 is similar but not what I was thinking), then does `readOnly` prevent a user profile from being layered by other profiles? * Do we throw a warning when the user tries to add properties in their settings to a readonly profile? (probably) * What happens if they mark `profiles.defaults` as readonly? (probably warn them, because that's an obvious footgun)
claunia added the Issue-FeatureArea-SettingsProduct-Terminal labels 2026-01-31 05:17:06 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#16606