-
Notifications
You must be signed in to change notification settings - Fork 29
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
Introduce s:opened_committa #17
Conversation
|
OK, I try implementing 『「現在すでに開いているか?」を知りたいなら現在のバッファのバッファ名を見ていって committia のバッファが見つかるかどうかで判定する』 |
description の追加ありがとうございます.読ませていただいた感じ,2回 open が呼ばれてしまう原因をつぶすほうが正攻法だと思うのですが,どうしてそうしないのでしょうか? |
#11 と同じで 他のプラグインとの干渉と思われるので, 今回の(別のプラグインでの)問題をみつけてもまた発生すると思い, committia.vim側で制御したいと思いました. こちらに PR を送ってしまったのはミスですが, 送ってしまったので適当にしたいと思います |
うーん,了解です.「すでに開いていたら再度開かないようにする」はチェック入れる意味が確かにありますね.説明ありがとうございます. |
の方向で実装しようとしてみましたが, 私の症状は s:open_multicolumn() が呼ばれたあとで s:open_singlecolumn() が呼ばれるため変更箇所が大きくなるので, 一旦あきらめました. 利用しているプラグインについて見直したところ Shougo/vimfiler のために |
ここのところ neobundle 周りで何件かうまく動かないと言われているので,最近の neobundle か vimfiler の変更が問題の可能性があります.(ちらっと聞いた話だと お手数ですが,vimfiler のほうに issue を立てていただくのが一番良いかもしれません. |
バッファ名のチェックをやりかけたものです. これだけだと s:open_multicolumn() が呼ばれたあとで s:open_singlecolumn() が呼ばれた際に s:current_info から status の情報が消えてしまうので うまくありません. 今日は時間がないので vimfiler の explorer の設定で再現するかどうかだけ調べて |
g:vimfiler_as_default_explorer を 1 にしていましたが 0 にしても挙動に変化はありませんでした. おつきあいありがとうございました |
いえ,そちらの explorer ではなく, https://github.com/Shougo/neobundle.vim/blob/master/doc/neobundle.txt#L1127 |
こちらの explorer を削除すると問題がなくなりました |
確認ありがとうございます.うーん,ちょっとこっちでも見てみて,場合によっては neobundle のほうに issue 立ててみます. |
In my environment, committia#open() is called twice after
git commit
.s:open_diff_window() is called twice and
setlocal nomodifiable
fails.This PR is not appropriate on g:committia_open_only_vim_starting = 0 environment.