search

keflavich / macvim-skim

Interlink macvim & skim for an integrated LaTeX DE
16 stars 6 forks source link

Switched to buffer and localleader #9

Closed tbelaire closed 8 years ago

tbelaire commented 8 years ago

I switched to using nnoremap, and only defining the commands for the current buffer.

Since ftplugins are only run once you know that the filetype is tex, go ahead an define the mappings unconditionally, but use <buffer> so that they only get loaded for the current (tex) buffer, not all open buffers.

Also, this is a filetype specific command, so it makes sense to use <localleader> (which I have mapped to , anyways), so now people who remap local leader are happy too.

Also, I split the commands up by line, that should make it easier to see what's going on. It also highlights some of the redundancy, perhaps there should be a function defined?

This branch also includes the exists check I fixed earlier, since I couldn't load the plugin at all with that constantly crashing. Maybe resolve that pull request first.

tbelaire commented 8 years ago

Oops, I don't want this to be from the master branch, since there's some more changes I want to make and I don't want to interfere with this. Reopening, sorry.