Skip to content

fix(input): support Shift+Tab reverse cycling#213

Merged
agavra merged 3 commits intoagavra:mainfrom
villads-valur:fix/upstream-shift-tab-reverse-cycling
Mar 20, 2026
Merged

fix(input): support Shift+Tab reverse cycling#213
agavra merged 3 commits intoagavra:mainfrom
villads-valur:fix/upstream-shift-tab-reverse-cycling

Conversation

@villads-valur
Copy link
Copy Markdown
Contributor

Summary

  • Add Shift+Tab (BackTab) support to reverse cycle comment types in Comment mode.
  • Add Shift+Tab reverse focus traversal in Normal mode to complement existing forward Tab focus cycling.
  • Update help text, comment panel hints, and README keybindings to document forward/backward Tab behavior.

Testing

  • cargo fmt --all
  • cargo test input::keybindings::tests
  • cargo check

};
}

pub fn cycle_comment_type_reverse(&mut self) {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this work with #211? I feel like it should be generic ordering

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, updated. now uses the same generic ordering as forward Tab cycling, so it works correctly with configurable comment types from #211.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated with generic ordering: reverse cycling now resolves current type index from the configured comment_types list and wraps to the last entry when needed (same model as forward Tab cycling).

@villads-valur villads-valur force-pushed the fix/upstream-shift-tab-reverse-cycling branch from 7018812 to 27c5877 Compare March 20, 2026 08:43
@agavra agavra merged commit 7ab9b99 into agavra:main Mar 20, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants