Skip to content

Conversation

@coot
Copy link
Contributor

@coot coot commented Jul 31, 2017

This enables:

  • gf normal command over module name in the import section
  • ]i, ]I normal commands
  • :find command to navigate through all the source files

This enables:
* gf normal command over module name in the import section
* ]i, ]I normal commands
* :find command to navigate through all the source files
@raichoo raichoo requested a review from arthurxavierx August 2, 2017 18:45
@raichoo
Copy link
Collaborator

raichoo commented Aug 2, 2017

I'm currently not actively working on this so maybe @Arthur-Xavier can also have a look.

@arthurxavierx
Copy link
Collaborator

These are some nice improvements there! Thanks a lot! ]i and ]I are fantastic!

Tested it all and everything works fine as of vim-8.0.525 and nvim-0.2.1-359. It's a shame, however, that imports under bower_components could only be located with gf after I removed bower_components from my wildignore list.

Do you know perhaps any solution for that?

@arthurxavierx arthurxavierx merged commit 5688098 into purescript-contrib:master Aug 4, 2017
@coot coot deleted the includeexpr branch August 4, 2017 00:22
@coot
Copy link
Contributor Author

coot commented Aug 4, 2017

vim normal mode commands are are linked to command, I think that gf is sharing code with find. But I couldn't reproduce your issue even with se wig+=*.purs neither with se wig=bower_components/** (and set acd=0).

What you could do is to map gf to a function which temporarily changes wildignore:

fun! Gf()
  let wig = &wig
  set wig -= bower_components
  normal "gf"
  let &wig = wig
endfun

@arthurxavierx
Copy link
Collaborator

arthurxavierx commented Aug 4, 2017

That's a witty trick. Works great! Unfortunately I had to use it as <leader>gf. Otherwise it would infinitely recurse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants