-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Can we bring the fugitive_autoreload_status
setting back?
#2207
Comments
There's now two kinds of reloading: eager and lazy. Eager immediately reloads any visible Fugitive windows, and is used when you explicitly invoke a Fugitive command that will likely change the status, like So it's no longer as simple as a Boolean option, that's why the old option went away. My proposal would be to add an option that makes all reloads lazy. I think if the Fugitive window is in focus, then we should still reload it, otherwise some operations won't work right. Does an option for that sound like it would solve or at least substantially mitigate your problem? |
Thanks for looking into this. So if the proposed option is on, it will not reload status after we invoke a Fugitive command unless the If yes, that should work as we could mitigate the issue by avoiding switching to the |
Best I can tell the ability to pass a line number or tab number argument has never been used. Cleaning this up in anticipating of adding actually useful arguments, and in anticipation of that, I'm preemptively making ReloadWinStatus() blindly relay any arguments to ReloadStatusBuffer(). References: #2207
The second reload was short-circuited by a reltime() check, so no big deal, but it does get in the way when trying to make the reload conditional. References: #2207
This should improve performance on large repositories. References: #2207
Have you tried I started working on bring the option back, but then quickly decided I'd rather spend what time I have working on making it async instead. This turned out to be a major project. |
Those git settings haven't made things better for me. I might haven't described the issue very well. It is acceptable to reload status when a Fugitive command is invoked with |
There is an option to turn off reloads on focusing Vim: let g:fugitive_focus_gained = v:false |
In some large repo, it takes minutes to run
git status
. Vim freezes frequently ifGit
window is open.Is it possible to turn off autoreload_status and let user to refresh status on demand?
This is some related setting
vim-fugitive/autoload/fugitive.vim
Line 2062 in 5d99841
The text was updated successfully, but these errors were encountered: