Skip to content
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

vim-blockle plugin #4

Merged
merged 1 commit into from
May 11, 2016
Merged

vim-blockle plugin #4

merged 1 commit into from
May 11, 2016

Conversation

ta1kt0me
Copy link
Contributor

rubyのブロックのsyntaxをtoggleするプラグインを入れました。ブロックの中で <Leader>b をすると、下の二つを切り替えてくれます。

# one line
foo.each { |e| e.bar }  

# multiple lines
foo.each do |e|
  e.bar
end

@5t111111
Copy link
Contributor

おおーこんなのもあるんですね!

<Leader>b は (今アサインされていないとはいえ) かなりファーストクラス感のあるキーバインドだけど Rubyist だと結構頻繁に使いそうなので良さげですね。

…と思ったけど、

  • 「one line」=> 「multiple lines」

の方はちゃんと動くんですが、

  • 「multiple lines」=> 「one line」

の切り替えが、色んなパターンで試してるけど1つもうまくいきません…コードがおかしくなってしまう。

そのサンプルのコードですらダメでした。使い方が間違ってるのかなあ。そちらでは問題ないですか?

@5t111111
Copy link
Contributor

お、まじか。neovim だとうまく動いた

@5t111111
Copy link
Contributor

自分の場合、neovim だと期待どおりに動くんだけど、普通の Vim だと以下のどっちもダメでした。

VIM - Vi IMproved 7.4 (2013 Aug 10, compiled May  6 2016 13:20:05)
MacOS X (unix) version
Included patches: 1-1817
Compiled by Homebrew
Huge version without GUI.  Features included (+) or not (-):
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled May  6 2016 13:20:05)
MacOS X (unix) version
Included patches: 1-1817
Compiled by Homebrew
Huge version without GUI.  Features included (+) or not (-):

てか、もう issue あがってますね。
jgdavey/vim-blockle#10

とりあえずまったく動かないわけじゃないのでひとまずマージして、どうしてもダメそうなら不具合のあるプラグインをずっと入れとくのもアレなので改めて外すことも検討する感じでいきましょう。

最悪がんばって直して PR 投げるか fork するのもアリかもですし、そういうののきっかけになるのもこのリポジトリの意義でもあると思うので。

ということで LGTM!

@5t111111 5t111111 merged commit d70c982 into master May 11, 2016
@5t111111 5t111111 deleted the install_vim-blockle branch May 11, 2016 16:05
@ta1kt0me
Copy link
Contributor Author

yochiyochirb/vimrcの設定だとうまく動かないですね...なにが悪さしてるんだろう。

@5t111111
Copy link
Contributor

あ、てことは設定によっては動くってことですね。結構プラグインぶつかるから、一度設定空の状態から切り分けしてみるしかないかもですね〜
時間あるときやります。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants