kristijanhusak / vim-packager

Vim plugin manager that utilizes "jobs" and "pack" features.
MIT License
245 stars 9 forks source link

Updating plugins does not move cursor to Packager window #28

Closed deathlyfrantic closed 4 years ago

deathlyfrantic commented 4 years ago

This wincmd p was added in commit 9ad1cd2 but I don't really understand why. Was this change intended? It looks like it might be left over from troubleshooting.

https://github.com/kristijanhusak/vim-packager/blob/e168eb5fc67fad2ab612ef2ed309b16a93b672fd/autoload/packager.vim#L280

kristijanhusak commented 4 years ago

Yeah it was intended. Idea was to not mess up the cursor position in case you're in middle of something. It is also a bit strange for me, so i'll probably remove it.

deathlyfrantic commented 4 years ago

I can work around it but it seems unintuitive. I probably wouldn't call packager#update() (or any of the other commands) if I were in the middle of something. 🤷‍♂️

kristijanhusak commented 4 years ago

@deathlyfrantic I fixed it. It should not happen any more.