-
Notifications
You must be signed in to change notification settings - Fork 32.5k
Edit object type settings from the settings GUI #99635
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
Comments
You'll want to explicitly add a Milestone to every feature-request or else the bot will auto-assign it to our |
@9at8 With this work, would it be possible to render enums values as a dropdown in the UI? For my specific use case, this would make the UX much better by preventing users from having to read external documentation. In an ideal case, the keys would be hard-coded in the UI for that setting, and the values would be dropdowns. |
@savannahostrowski yes! |
I'm working on making simple non-nested object type settings editable from the settings GUI in vscode (microsoft/vscode#99635), and I think that setting `"additionalProperties": false` for the objects that don't expect more properties would be great!
When I'm working on making simple non-nested object type settings editable from the settings GUI in vscode (microsoft/vscode#99635), and I think that setting `additionalProperties` to `false` for the objects that don't expect more properties would be great! As per microsoft/vscode#101810, vscode (v1.48) will hide the "Add item" button if and only if `additionalProperties` is `false` and all known properties are present. Change-Id: I92380610b24f7cd6ee790f2e020b9c689c3ff25b GitHub-Last-Rev: 695aa78 GitHub-Pull-Request: #284 Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/240907 Reviewed-by: Hyang-Ah Hana Kim <[email protected]>
A lot of settings expose configuration in the form of simple non-nested objects. It will be useful to explore different ways to expose editing functionality for these kinds of objects in the GUI.
Non-nested objects can be broadly put in three different categories:
properties
keyword and default values for all properties are defined.patternProperties
keyword andadditionalProperties
keyword, and NOT theproperties
keyword.properties
keyword, and at least one ofpatternProperties
keyword andadditionalProperties
keyword.Requirements ✅
Static Objects
Dynamic Objects
additionalProperties
/patternProperties
typeSome unknowns 🤔
boolean
, then we should render checkboxes, just like everywhere else in the settings UI.boolean
values and other primitives?Impacted Settings ⚙️
files.associations
emmet.includeLanguages
emmet.variables
This is not a complete list. Settings that have type
object
and define eitherproperties
,patternProperties
oradditionalProperties
as a non-nested object might be impacted.Prototypes
You can play around with the setting by using the insiders build of VSCode.
Latest Prototypes (in insiders)
Example from the go extension:
Example:
files.associations
Old prototypes
Example from the go extension:
Example:
files.associations
cc: @roblourens
The text was updated successfully, but these errors were encountered: