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

jumplist: Add documents to view history #3593

Merged

Conversation

the-mikedavis
Copy link
Member

This change adds documents to the view's document history Vec.
(This is used by ga for example to access the last buffer.)

Previously, a sequence like so would have confusing behavior:

  1. Open file A: any document with an active language server
  2. Find some definition that lives in another file - file B - with gd
  3. Jump back in the jumplist with C-o to file A
  4. Use ga intending to switch back to file B

The behavior prior to this change was that ga would switch to file
A: you could not use ga to switch to file B.

This change adds documents to the view's document history Vec.
(This is used by `ga` for example to access the last buffer.)

Previously, a sequence like so would have confusing behavior:

1. Open file A: any document with an active language server
2. Find some definition that lives in another file - file B - with `gd`
3. Jump back in the jumplist with `C-o` to file A
4. Use `ga` intending to switch back to file B

The behavior prior to this change was that `ga` would switch to file
A: you could not use `ga` to switch to file B.
@archseer archseer added this to the 22.08 milestone Aug 29, 2022
@archseer archseer merged commit 701cea5 into helix-editor:master Aug 31, 2022
@the-mikedavis the-mikedavis deleted the md-jumplist-add-docs-to-history branch August 31, 2022 01:46
thomasskk pushed a commit to thomasskk/helix that referenced this pull request Sep 9, 2022
This change adds documents to the view's document history Vec.
(This is used by `ga` for example to access the last buffer.)

Previously, a sequence like so would have confusing behavior:

1. Open file A: any document with an active language server
2. Find some definition that lives in another file - file B - with `gd`
3. Jump back in the jumplist with `C-o` to file A
4. Use `ga` intending to switch back to file B

The behavior prior to this change was that `ga` would switch to file
A: you could not use `ga` to switch to file B.
jdrst pushed a commit to jdrst/helix that referenced this pull request Sep 13, 2022
This change adds documents to the view's document history Vec.
(This is used by `ga` for example to access the last buffer.)

Previously, a sequence like so would have confusing behavior:

1. Open file A: any document with an active language server
2. Find some definition that lives in another file - file B - with `gd`
3. Jump back in the jumplist with `C-o` to file A
4. Use `ga` intending to switch back to file B

The behavior prior to this change was that `ga` would switch to file
A: you could not use `ga` to switch to file B.
herkhinah pushed a commit to herkhinah/helix that referenced this pull request Dec 11, 2022
This change adds documents to the view's document history Vec.
(This is used by `ga` for example to access the last buffer.)

Previously, a sequence like so would have confusing behavior:

1. Open file A: any document with an active language server
2. Find some definition that lives in another file - file B - with `gd`
3. Jump back in the jumplist with `C-o` to file A
4. Use `ga` intending to switch back to file B

The behavior prior to this change was that `ga` would switch to file
A: you could not use `ga` to switch to file B.
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.

None yet

2 participants