Skip to content
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

Refactor goto_ts_object_impl as a motion #3264

Conversation

the-mikedavis
Copy link
Member

@the-mikedavis the-mikedavis commented Jul 30, 2022

This refactor changes the overall structure of the goto_ts_object_impl
command without removing any functionality from its behavior. The
refactored motion:

  • acts on all selections instead of reducing to one selection
  • may be repeated with the repeat_last_motion (A-.) command
  • informs the user when the syntax-tree is not accessible in the current buffer

Ultimately this moves the goto_ts_object_impl function closer to the goto_para_impl function and is part of (but kinda unrelated to) adding extend variants (#3263)

Closes #1993

This refactor changes the overall structure of the goto_ts_object_impl
command without removing any functionality from its behavior. The
refactored motion:

* acts on all selections instead of reducing to one selection
* may be repeated with the `repeat_last_motion` (A-.) command
* informs the user when the syntax-tree is not accessible in the current buffer
@archseer archseer merged commit 83f177d into helix-editor:master Aug 31, 2022
@the-mikedavis the-mikedavis deleted the md-refactor-goto_ts_object_impl-as-a-motion branch August 31, 2022 12:09
thomasskk pushed a commit to thomasskk/helix that referenced this pull request Sep 9, 2022
This refactor changes the overall structure of the goto_ts_object_impl
command without removing any functionality from its behavior. The
refactored motion:

* acts on all selections instead of reducing to one selection
* may be repeated with the `repeat_last_motion` (A-.) command
* informs the user when the syntax-tree is not accessible in the current buffer
jdrst pushed a commit to jdrst/helix that referenced this pull request Sep 13, 2022
This refactor changes the overall structure of the goto_ts_object_impl
command without removing any functionality from its behavior. The
refactored motion:

* acts on all selections instead of reducing to one selection
* may be repeated with the `repeat_last_motion` (A-.) command
* informs the user when the syntax-tree is not accessible in the current buffer
herkhinah pushed a commit to herkhinah/helix that referenced this pull request Dec 11, 2022
This refactor changes the overall structure of the goto_ts_object_impl
command without removing any functionality from its behavior. The
refactored motion:

* acts on all selections instead of reducing to one selection
* may be repeated with the `repeat_last_motion` (A-.) command
* informs the user when the syntax-tree is not accessible in the current buffer
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.

Repeat last motion doesn't apply to "next function"
2 participants