Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Policy and workspace trust can switch to using Setting indicators #157564

Closed
4 tasks done
rzhao271 opened this issue Aug 8, 2022 · 2 comments
Closed
4 tasks done

Policy and workspace trust can switch to using Setting indicators #157564

rzhao271 opened this issue Aug 8, 2022 · 2 comments
Assignees
Labels
config VS Code configuration, set up issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan settings-editor VS Code settings editor issues
Milestone

Comments

@rzhao271
Copy link
Contributor

rzhao271 commented Aug 8, 2022

Currently in the GUI Settings editor, settings affected by policies or untrusted workspaces display large alerts that also disallow the user to change the setting value.
Instead, we can consider moving those alerts into the indicators area, which is the area around each setting where indicators such as "Also modified in" can show up.

Done:

  • Change the configurationService to return the values for all scopes even for policy and workspace restricted settings, while keeping the actual inspected value the same as before. @sandy081
  • Change the UI to use indicators for policies and workspace trust. @rzhao271

Todo:

  • Get some rounds of feedback
  • Write TPIs for policy and workspace trust settings in the Settings editor
@rzhao271 rzhao271 added this to the August 2022 milestone Aug 8, 2022
@rzhao271 rzhao271 self-assigned this Aug 8, 2022
@rzhao271 rzhao271 added feature-request Request for new features or functionality settings-editor VS Code settings editor issues labels Aug 8, 2022
@rzhao271 rzhao271 added the config VS Code configuration, set up issues label Aug 15, 2022
@rzhao271 rzhao271 modified the milestones: August 2022, September 2022 Aug 23, 2022
@rzhao271 rzhao271 modified the milestones: September 2022, October 2022 Sep 26, 2022
@rzhao271
Copy link
Contributor Author

\closedWith 740d837

@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Oct 14, 2022
@rzhao271
Copy link
Contributor Author

Adding a link to the epic #147756

rzhao271 added a commit that referenced this issue Nov 3, 2022
Ref #165169
Ref #157564

After changing workspace trust and policy settings
to use indicators, a bunch of CSS rules became redundant.
This PR removes those CSS rules.
rzhao271 added a commit that referenced this issue Nov 3, 2022
Ref #165169
Ref #157564

After changing workspace trust and policy settings
to use indicators, a bunch of CSS rules became redundant.
This PR removes those CSS rules.
@github-actions github-actions bot locked and limited conversation to collaborators Nov 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
config VS Code configuration, set up issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan settings-editor VS Code settings editor issues
Projects
None yet
Development

No branches or pull requests

3 participants