Closed akinomyoga closed 5 years ago
PRありがとうございます。 あまり考えた事なかった部分ですが、使うケースを考えると、 ばらばらになった選択部分をまとめる、 みたいなことならこの機能の方が良いですね。
逆に今の機能を使いたい場面、というのが思いつかなかったので採用させていただきます。
また、この PR では v によって選択範囲を指定している途中 (_s_visual>=0) で C-i/C-o を押した時は、選択範囲の指定を其処で中断するというようにしました。
そうですね、こちらの方が使いやすいと思います。
ところでこの PR では ((_s_visual!=-1)) などのような算術式 (bash, zsh の両方で利用可能) を用いていますが、もし [[ $_s_visual -ne -1 ]] の様な形式を用いる理由があるようでしたら修正いたします。
単に他の比較のところで[[]]
を使っているので条件式をすべて[[]]
にしていただけです。
算術式に限れば(())
でも問題ないですしパフォーマンス的にも良いので
使わない理由はないです。
統一するために他の部分も変更しようと思います。
ありがとうございます! 失礼いたします
たびたび失礼いたします。これは提案なのでございますが、複数選択をしている時の C-i/C-o による項目の移動において、選択状態も一緒に移動すると良いのではないかという様に考えるのです。
_s_visual>=0
) で C-i/C-o を押した時は、選択範囲の指定を其処で中断するというようにしました。((_s_visual!=-1))
などのような算術式 (bash, zsh の両方で利用可能) を用いていますが、もし[[ $_s_visual -ne -1 ]]
の様な形式を用いる理由があるようでしたら修正いたします。お忙しいところすみませんが、ご検討いただければ幸いです。