Balanced fix for snippet Tab conflict (Issue #58)#59
Open
albertromkes wants to merge 1 commit intomasterfrom
Open
Balanced fix for snippet Tab conflict (Issue #58)#59albertromkes wants to merge 1 commit intomasterfrom
albertromkes wants to merge 1 commit intomasterfrom
Conversation
Owner
Author
SummaryThis PR introduces a balanced fix for the snippet-mode Tab conflict reported in #58, while preserving existing TabOut behavior for users who prefer it. What changed
Why this is balanced
Validation
Closes #58 |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary\n\nThis PR introduces a balanced fix for the snippet-mode Tab conflict reported in #58, while preserving existing TabOut behavior for users who prefer it.\n\n### What changed\n\n- Added new setting: \n - (default):\n - In snippet mode, try native snippet placeholder navigation first ().\n - If snippet navigation does not move the cursor, TabOut behavior applies as fallback.\n - :\n - Always prioritize existing TabOut behavior, including in snippet mode.\n\n- Updated command implementation to support the behavior above.\n- Added/updated keybinding config tests to cover the new setting and enum values.\n\n## Why this is balanced\n\n- Fixes #58 by default: snippet placeholder flow is respected.\n- Avoids degrading existing workflows: users who rely on TabOut-first behavior in snippets can switch to .\n- Non-snippet behavior is unchanged.\n\n## Validation\n\n- TypeScript compile: