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

[BUG] Does not play well with stevearc/aerial.vnim #56

Closed
joehannes opened this issue Dec 30, 2021 · 5 comments
Closed

[BUG] Does not play well with stevearc/aerial.vnim #56

joehannes opened this issue Dec 30, 2021 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@joehannes
Copy link

joehannes commented Dec 30, 2021

Describe the bug
when triggering the default mapping "gpd", instead of previewing the definition,
nvim is throwing an error regarding another plugin, namely stevearc/aerial.nvim

To Reproduce
Steps to reproduce the behavior:

  1. Install aerial.nvim alongside or clone my config after safeguarding yours
  2. I don't tweak those a lot, so default settings should suffice
  3. try to "gpd" on some usage of something ... (eg.: with cursor on the import of a React Component, in normal mode, press "gpd")
  4. observe neovim throwing the above error ...

Expected behavior
I would like to see the actual preview of the definition of the import/thingy

Screenshots
EF9CE19A-ACF5-4312-B00B-5C9CCD0C4403

Baseline (please complete the following information):

Additional context
Justy a quick note: I dblchecked on the default mappings of aerial, and "gpd" doesn't seem to be one of them

@joehannes joehannes added the bug Something isn't working label Dec 30, 2021
@rmagatti
Copy link
Owner

Hey @joehannes goto-preview works by creating floating windows, I have never used aerial.nvim but from the looks of it this is probably an issue to open with that repo instead of goto-preview, at least that's what it looks to me at first glance. I will dig a bit further when I get a chance.

@joehannes
Copy link
Author

hey @rmagatti :-) cool ... my thought was, that because it seems aerial doesn't have anything bound to "gpd" ... and the bug happens on "gpd" ... I thought while "goto-preview" is checking the buffers for whatever reason ... and so on ... but since you already say it's based on floating windows ... however: if it was due to the origin of the or something like that ...??? looking things up in a buffer there and somehow the current buffer would boil down to a hidden aerial buffer or something .. (because strangely enough, I don't even have the aerial outline opened when I get that err).
I'll ping the guys at aerial.nvim as well to let them know

@joehannes
Copy link
Author

hehey :-)

just to let you guys know, stevearc from aerial.nvim did some defensive coding and it solved the issue ...

@rmagatti
Copy link
Owner

rmagatti commented Dec 31, 2021

Got it. One thing @stevearc said that stood out to me was the buffer being deleted part. Pretty sure goto-preview isn't deleting it but depending on your vim settings, like if you don't have set hidden, buffers would be deleted automatically upon being unloaded. This might explain what's happening. Keep in mind this is just a guess for now.

@stevearc
Copy link

Yeah, I wouldn't assume that it's any sort of problem with goto-preview. Likely as you said, just some interaction with hidden or bufhidden. You're probably safe to close this out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants