Closed vhakulinen closed 4 years ago
Yup, still WIP. Good to know that the floats crash, I don't really have any test cases for them (floats) on my own workflow currently.
Rebased.
Let me know when/if this is ready for QA :+1:
Things should be now ready for testing. Some minor bugs still, like the messages window showing things from earlier renders if the grid size is not even and I noticed that fzf
window has blinking cursor even tho' it also has a solid one (probably something to do with the terminal mode). Also, after a window split there might be some black lines/areas because of a bug in nvim.
Awesome! Thanks for the heads up, will test along this week.
The floating window position overlaps with the autocompletion using coc.vim (both when hint is above and below the cursor):
I'm not sure yet how to reproduce it, but after switching to a tab, only half the buffer is redrawn.
This is my current blocker on continuing with tests.
coc won't be able to figure out the position for the floating window because of the externalized popupmenu. You'll either have to add logic to gnvim to expose the popupmenu position to nvim (and thus' to coc) or disable the externalized popupmenu.
You'll either have to add logic to gnvim to expose the popupmenu position to nvim . . .
@badosu I'll ask about this on the Neovim Gitter, and if I come up with something I'll PR it and let you know.
@vhakulinen @smolck Thanks for checking, in neovim the positioning works fine.
I am not sure if this is an issue in neovim for this particular case (having externalized popupmenu and the completioninfo not interacting well due to lack of information on protocol), or something that neovim expects the GUI to deal with outside neovim's scope.
I'm not sure yet how to reproduce it, but after switching to a tab, only half the buffer is redrawn.
This is my current blocker on continuing with tests.
By the way, the issue above is a separate one and happens after less than 10 minutes of usage. I'm fine with the popupmenu one, but this one is a blocker.
I am not sure if this is an issue in neovim for this particular case (having externalized popupmenu and the completioninfo not interacting well), or something that neovim expects the GUI to deal with outside neovim's scope.
@badosu After asking @bfredl on the Neovim Gitter about this, it seems it would take a (apparently not all that difficult) PR to add relative='popupmenu'
to the floating window API, which if I understand correctly would fix this issue.
Note to self: somewhere is a integer underflow. It happened while the msg window was constantly resizing (was doing %s/something/something
).
Also, in FZF window the cursor is duplicated because the terminal buffer has its own solid cursor and gnvim is drawing its own cursor (and thats one column behind).
I just tested again and the behavior where after dealing with splits, opening tabs, etc creates a bug where part of the buffer gets hidden is still present.
I'm not sure if this is related to FZF (I'll test it again without vimrc) but I use it extensively to handle file opening.
I just tested this very briefly. most things regarding floats and external windows seems to be working. But when I use call nvim_win_set_config(0, {'external': v:false, 'width':50, 'height':10})
i e convert an existing split window to an external window, the size of the window is wrong.
Good to know :+1:
Ugh. This code is quite messy and I don't remember enough of the details to clean it up, at least in timely manner... Anyways, this seems to be somewhat working now. Any tests cases for floating/external windows are more than welcome, if someone happens to have any.
This is now ready for testing. Going to merge soon.
Waiting for https://github.com/neovim/neovim/pull/12612.
Awesome, I've been using a thin wrapper instead of gnvim after I unfortunately got the blur bug because of an update. Happy to test as soon as the PR is merged
I've been using a thin wrapper instead of gnvim after I unfortunately got the blur bug because of an update.
The blurry scrolling thing related to pango 1.44 is fixed in master.
Rebased.
Great progress here :) Message area already works well, but it seems to crash very quickly on floats, even something simple as
Or is this simply still WIP?