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

Rename in File Browser opens newly renamed file in current tab. #72

Open
jordwalke opened this issue Jan 9, 2013 · 3 comments
Open

Comments

@jordwalke
Copy link

Renaming opens the file. I don't think it should. Is this correct?

@alloy
Copy link
Owner

alloy commented Jan 9, 2013

It should really only re-open the renamed file if the old file was open. If it wasn’t opened to begin with, then the renamed file should not be opened either.

There might be some subtle issues here with a buffer existing vs it being open in a tab, though.

@jordwalke
Copy link
Author

So in my case, it was open in another tab, in a particular vertical split. However, I believe it did something very wrong. It closed my current tab 9, and clobbered the next tab 8 with the contents of the renamed file. The whole time the file was already opened in tab 1. I think it might be easiest to just disable to (attempting to open file on rename) behavior and instead just simply refresh any existing buffers/windows that have that file open. Could you point me to where this happens in the code?

I believe it also removed the focus from both the file browser and the main code editing window.

@jordwalke
Copy link
Author

However, I really do appreciate the fact than any time I select a file in the left nav, it finds the existing window/split/tab with that file name. That's really awesome and a must for any modern editor.

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

No branches or pull requests

2 participants