-
Notifications
You must be signed in to change notification settings - Fork 8
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
ブロックキーが入力されるまでインプットを保持するオプションを追加 #11
base: master
Are you sure you want to change the base?
Conversation
こんにちわ。
|
コメントありがとうございます。
|
取り込むかどうかは、@rhysd さんの判断なので、阻害するつもりはないのですが。
vim-operator-furround でできることをご存じないと予想したのでお知らせしただけなんですが。
「それ」はどれを指していますか?
ブロックキーの設定を使っている人も使えるようになればより良いのではないですか? -> パッチ改善の余地 |
「稀」なんて書いていません。
どちらかというと後者に近いですが、極端に短いブロックキー設定をしている場合にこの設定との相性は悪い、というレベルと思います。
こちらについてはどのような意見でしょうか? |
プルリクエストありがとうございます. すでに vim-operator-furround で同様の機能は提供されていますが,本家にあっても良いと考えているので,マージする方向でやりたいです. いくつか気になった点を上記議論踏まえて
これが理解できていないのですが,言及されているという記事の該当箇所がどこかを教えていただけませんでしょうか. ちょっと今日は試す時間が無いため,後日私の手元でも動作確認してみます. |
回答ありがとうございます。前向きに検討頂けてすごく嬉しいです。
コードレビュー頂いた中で一つ認識齟齬がありそうです。
すみません、言及箇所は私もよくわかっていません。 |
これの意図は, @zeero さんの説明の通りです。
言及しているのは、上記の部分のみをさしているつもりでした。
(後者の場合に)統一したいだけなら,
稀, 失礼しました。 surround.vim の設定例: https://github.com/t9md/vim-surround_custom_mapping 例えば、xbrackes-mode の入力待ちなら、デフォルトのカッコしか使わないとかになってくれると嬉しいかも。それでも、 |
いったんコードレビューの指摘とオプション名の変更に対応しました。ドキュメントについてはもう少し書きっぷりを変える必要がある認識ですが取り急ぎ。
その場合せっかく設定できるブロックキー設定が意味を為さなくなるのではないでしょうか。オプションはコロコロと切り替えるようなものでもないと思いますので。 |
前提を無視するからでしょう. 私はこの機能をいまの前者を無視した仕様で取り込むべきではないと思っています.
ごめんなさい. だいぶ説明がおかしいですね. |
ええっと,理解力が無くて申し訳ないのですが,上記のコメントを全部読んだ上で理解していないので,そうコメントされてもやはり理解できません. { 'block' : ['hoge(', ')'], 'motionwise' : ['char', 'line', 'block'], 'keys' : ['a'] } 正直,アルファベット1字とかは想定していないので何とも言えないところはありますが,その辺りはユーザのほうで調整してほしいなぁと思います. これはブロックを入力した時点で確定される仕様になっている以上仕方ない気がしているのですが,何か良い案あるでしょうか? ちなみに,もし僕の解釈がまだ間違っている場合は,困る場合の設定例を Vim script で書いていただけると一番助かります. @syngan さんがある程度納得できる案で摺り寄せて,僕なり @zeero さんが実装修正して最終的に取り込む感じでいきたいです.
この点は異論ないです. |
そうですね。私は mode=2 なら使わないと思います.
あっています。説明が下手ですいません。
他に私が考えていたのは prefix を付ける方法ですが、これだと @zeero さんの要望とあわないので避けていました。 https://gist.github.com/syngan/9949509 これなら、以下の設定( { 'block' : ['hoge(', ')'], 'motionwise' : ['char', 'line', 'block'], 'keys' : ['a'] }
|
「surround.vim は一文字しか設定できなかった」ので、その設定のまま移行すると一文字なのが一番自然だと思っていました。
私の設定が「極端である」が真であるなら、私の仮定がおかしい突込みなので無視してもらって構わないです。 ところで、この機能っていつ使われるか @zeero さんはイメージされていますか? |
コメントありがとうございます.返信遅くてすみません.
提案ありがとうございます.本機能は「普通に関数名を入力して,その後引数部分を自分で囲む」のに比べてどれくらい便利かが勝負だと考えています.最初の提案では「関数入力→引数を囲む」が1ステップで出来ていたので普通に入力する場合に比べて良いなぁと思っていたのですが,そのメリットが潰れてしまうのがつらいところですね…
極端かどうかは程度問題と使い方次第な気がするので,そこをあまり議論しても仕方ない気もします…
なるほど,参考になります.そうすると, |
コードまでよく見れていないのですが,その下の説明を読む感じ, |
返事遅くなっていてすみません。 極端かどうかについては@rhysdさんと同意見です。
初めに書いた通り、特にVimScriptを書いているときに便利だと思っています。私の場合はすでに手放せないレベルです。 そう考えると、 |
考慮すべきかどうかで重要かと思っていたんですが、んじゃ了解です。
ぬ。これは
一文字というか、入力の頭でマッチしない場合はユーザ定義を使用しない感じです.
りんだんさんも zeero さんも私が思っているのと違う使い方を想定されているんですね。。 お二人の使用イメージだと、関数で囲ったあとカーソル位置は閉じかっこの場所に移動するほうが自然なんでしょうか. 仮に
|
上記を |
こんにちは。素晴らしいプラグインをありがとうございます。
surroundを使うときにブロック以外の文字を足したい時があります。
vimスクリプトを書いていると処理の順と記述の順が逆なので特にそう思います。
(例)ファンクションAを呼んで、結果にファンクションBを呼びたい
このためにオプションを一つ足してみました。
g:keeps_input_if_no_block
chars & block :
に変わります。g:uses_input_if_no_block
を無視することになるので、既存の動きを変えないほうがいいかと思いました。VACの hoge() で囲みたい症候群 - vim 初心者の作業メモ を読んで、
同じようなことを思っている人がいるなら、オプションでできるようになれば便利かなと思った次第です。
ご検討よろしくお願いします。