stevearc / aerial.nvim

Neovim plugin for a code outline window
MIT License
1.55k stars 76 forks source link

bug: `51a0794` do not respect aerial position #369

Closed Zeioth closed 1 month ago

Zeioth commented 1 month ago

Neovim version (nvim -v)

NVIM v0.9.5 Release

Operating system/version

arch

Output of :AerialInfo

Aerial Info                                                                                                                                                                  
-----------                                                                                                                                                                  
Filetype: neo-tree                                                                                                                                                           
Configured backends:                                                                                                                                                         
  lsp (not supported) [No LSP client found that supports symbols]                                                                                                            
  treesitter (not supported) [No treesitter parser for neo-tree]                                                                                                             
  markdown (not supported) [Filetype is not markdown]                                                                                                                        
  man (not supported) [Filetype is not man]                                                                                                                                  
Show symbols: Constructor, Enum, Function, Interface, Method     

Describe the bug

See post below

What is the severity of this bug?

minor (annoyance)

Steps To Reproduce

Expected Behavior

More info

Relevant commit https://github.com/stevearc/aerial.nvim/commit/51a07949abf169b4cad30e14c165ac1ec0ce4e6f

Zeioth commented 1 month ago

Opening aerial while the cursor is over NeoTree, open aerial in the wrong position.

screenshot_2024-05-01_18-20-08_987306721

If I open it while the cursor is over the buffer, it opens on the right position (on the right) screenshot_2024-05-01_18-20-15_002572586

Zeioth commented 1 month ago

If you have no time to look into this, reverting the commit would be enough to at least prevent users from breaking the UI.

The mentioned commit seems to break ignore = {} too

Zeioth commented 1 month ago

@stevearc I've tested the latest commit and all looks good thank you so much. I'm closing the issue too.