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

":GV": consider opening tab *before* current tab #19

Closed
justinmk opened this issue Mar 6, 2016 · 4 comments
Closed

":GV": consider opening tab *before* current tab #19

justinmk opened this issue Mar 6, 2016 · 4 comments

Comments

@justinmk
Copy link
Contributor

justinmk commented Mar 6, 2016

In fugitive's :Gstatus window, cvc/cva opens its tab before the current tab, so that closing the tab returns to the correct original tab. When I q out of a :GV tab, it returns to the wrong tab. Any opposition to following fugitive's behavior?

@junegunn
Copy link
Owner

junegunn commented Mar 6, 2016

No objections. I just wasn't aware of the inconvenience since I rarely use tabs.

@junegunn
Copy link
Owner

junegunn commented Mar 6, 2016

Fixed it. One thing I noticed though is that O mapping from fugitive in git buffer opens a new tab after the current split. Seems a bit inconsistent.

@justinmk
Copy link
Contributor Author

justinmk commented Mar 6, 2016

Good point. I imagine the justification is that cvc and cva are intended as "short-lived" tabs, for which the common case is that the user closes them and most likely wants to return to the original tab. I'd say similar logic applies to :GV. And I'd even say it applies to fugitive's O in most cases...

But since fugitive's O is documented as :Gtabedit (which surely must match the behavior of Vim's :tabedit), maybe tpope didn't want to bother explaining the difference. And after this insanity I doubt he would entertain changing the existing behavior.

Sadly this means I can't drop the idea that Vim needs a "previous tab" feature.

Also, thanks!

junegunn added a commit to junegunn/fzf that referenced this issue Mar 6, 2016
@junegunn
Copy link
Owner

junegunn commented Mar 6, 2016

after this insanity

Haha, that's crazy.

I also applied the change to the neovim plugin of fzf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants