vim-dirvish dont use internal function for mapping

dont use internal function for mapping

Open bounceme opened this issue 7 years ago • 4 comments

https://github.com/justinmk/vim-dirvish/blob/e4b3c4b62d0ddb2d24aa8f9fcf3820f6d6de4ebf/ftplugin/dirvish.vim#L20

https://github.com/bounceme/remote-viewer/blob/master/plugin/remotedir.vim#L88

It makes it simpler to have :Dirvish used there for the external scripts like the one above ^

Jan 05 '18 04:01 bounceme

Is that going to work for a and o?

Jan 05 '18 18:01 justinmk

so a|o is basically: split, make dir listing buffer without using cache?

Jan 05 '18 21:01 bounceme

what about: https://github.com/justinmk/vim-dirvish/blob/e4b3c4b62d0ddb2d24aa8f9fcf3820f6d6de4ebf/ftplugin/dirvish.vim#L23

changed to (something like): execute 'nnoremap '.s:nowait.'<buffer><silent> o :<C-U>exe "split +edit\\" fnameescape(getline("."))<bar>wincmd p<cr>'

Jan 06 '18 05:01 bounceme

That won't work if multiple files are selected, specially if any fail to open.

I think adding hooks to override behavior would make more sense.

Jan 06 '18 14:01 justinmk