-
Notifications
You must be signed in to change notification settings - Fork 19
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
Implement conditional bindings #14
Merged
Merged
Changes from 24 commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
e184d3c
Support passing of "when" with key listener
stevenguh 2ba3e9e
Implement conditionals type
stevenguh ba48ad4
Add ConfigSections type
stevenguh 0cd6fb6
Add optional name in ConditionalBindingItem
stevenguh be2d670
Refactoring to a inheritance model for menu items
stevenguh 610b20b
Traverse through override bindings with keys
stevenguh a6d6735
Remove useless statment
stevenguh 7eecab6
Implement query-string like key as condition
stevenguh 8899e23
Simplify menu selection code
stevenguh c920f82
Add async to onDidAccept
stevenguh e08aae7
Fix broken transient
stevenguh eeee24a
Use vscode api for binding this for callbacks
stevenguh 84e8dea
Use ; instead , as separator of command
stevenguh d56d1bf
Small refactoring for error rasing
stevenguh 34fa407
Refactoring the ConditionalMenuItem
stevenguh 203a36d
Fix the bug where undefined key are not recognized
stevenguh ba32689
Show status bar msg when no conditionals matched
stevenguh d91efe8
Update doc for conditional bindings
stevenguh cd8398d
Remove an extra layer of await function
stevenguh 533959a
Update wording
stevenguh ebb8491
Code cleanup
stevenguh 62f2216
README: remove trailing whitespaces
marcoieni 758a45e
README: add some missing new lines
marcoieni 59a747d
README: fix grammar, improve readability
marcoieni a798b2b
Clarify conditional section of the README
stevenguh 1d6b6b4
README: fix typo, simplify sentences
marcoieni 11a0c62
README: hide experimental feature by default
marcoieni File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This and the following json are not very clear to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated, let me know the updated version is clearer :)