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

Fix 2 issues: (1) buffer that we switched to remains unlisted; (2) problem with symlink buffers #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

slycelote
Copy link

This reverts commit 8a8fa09 ("Switch buffer if file is already opened instead of always call edit.")

The problem that it is supposed to address ("the cursor always returns
at the beginning of the file") is not specific to fswitch and can be
fixed by an autocommand in a more generic way (autocmd BufWinEnter * exe "normal! g""`).

On the other hand, this commit introduces two problems:

  • when switching to an unlisted buffer, it remains unlisted. This
    leads, for example, to counterintuitive behavior in buftabline (and
    probably other similar plugins): currently opened buffer is not
    in buffer list.
  • problem with opening symlink buffers (Fix problem with opening symlink buffers #11)

Reverting it solves both.

…l edit."

This reverts commit 8a8fa09.

The problem that it is supposed to address ("the cursor always returns
at the beginning of the file") is not specific to fswitch and can be
fixed by an autocommand in a more generic way (autocmd BufWinEnter * exe
"normal! g`\"").

On the other hand, this commit introduces two problems:
  - when switching to an unlisted buffer, it remains unlisted. This
    leads, for example, to counterintuitive behavior in buftabline (and
    probably other similar plugins): currently opened buffer is not
    in buffer list.
  - problem with opening symlink buffers (derekwyatt#11)

Reverting it solves both.
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.

1 participant