Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update Select Configurations command to offer Edit Configs JSON option#3519

Merged
Colengms merged 3 commits into
mastermicrosoft/vscode-cpptools:masterfrom
coleng/SelectConfigurationEditUseJsonmicrosoft/vscode-cpptools:coleng/SelectConfigurationEditUseJsonCopy head branch name to clipboard
Apr 24, 2019
Merged

Update Select Configurations command to offer Edit Configs JSON option#3519
Colengms merged 3 commits into
mastermicrosoft/vscode-cpptools:masterfrom
coleng/SelectConfigurationEditUseJsonmicrosoft/vscode-cpptools:coleng/SelectConfigurationEditUseJsonCopy head branch name to clipboard

Conversation

@Colengms

Copy link
Copy Markdown
Contributor

The "Select Configuration..." command will display a list of configurations to choose between, or an option to "Edit Configurations...".

Settings UI uses this Select Configurations command to allow the user to change which configuration is currently being edited.

Because the user is likely to select the option to edit configurations when the configuration they want to select is not on the list, and because settings UI does not yet allow new configurations to be added, we should offer "Edit Configurations (JSON)" in the selection list instead.

Comment thread Extension/src/LanguageServer/ui.ts Outdated
@Colengms Colengms merged commit cfc908a into master Apr 24, 2019
@Colengms Colengms deleted the coleng/SelectConfigurationEditUseJson branch April 24, 2019 18:19
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.