Skip to content

Reimplement hover window with floating window for Neovim 0.4.0 or later #767

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.

Sign up for GitHub

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

Merged
autozimu merged 19 commits into from
Mar 26, 2019
Merged

Reimplement hover window with floating window for Neovim 0.4.0 or later #767

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
572f84a
Reimplement hover window with floating window for Neovim 0.4.0 or later
rhysd Mar 12, 2019
db88b8e
Add test for hover window using floating window
rhysd Mar 14, 2019
1a7748f
Close floating window when entering other window on WinEnter
rhysd Mar 14, 2019
d63389f
Add test for closing floating window on WinEnter
rhysd Mar 14, 2019
5ccf450
Fix checking enough space for floating window
rhysd Mar 14, 2019
eb752f2
Use BufEnter instead of WinEnter
rhysd Mar 16, 2019
e240734
Add test where entering another buffer closes floating hover
rhysd Mar 16, 2019
65ccc70
Follow the API change of nvim_open_win()
rhysd Mar 17, 2019
05afc80
Move cursor into floating hover when hover is already open
rhysd Mar 21, 2019
45f6d06
Add health check for floating window
rhysd Mar 21, 2019
04dce37
Describe the behavior of floating window support in document
rhysd Mar 21, 2019
0fbded1
Add g:LanguageClient_useFloatingHover
rhysd Mar 21, 2019
372939d
Add test for moving the cursor into floating window
rhysd Mar 21, 2019
f40a1c3
Add LanguageClient#openHoverInSeparateWindow()
rhysd Mar 21, 2019
18640c2
Rename LanguageClient#openHoverInSeparateWindow to LanguageClient#reo…
rhysd Mar 21, 2019
6f4ce81
Give left margin only when line is not empty
rhysd Mar 21, 2019
4c14bd4
Revert "Add LanguageClient#reopenHoverInSeparateWindow()"
rhysd Mar 23, 2019
79e36ba
Prefer s:GetVar() to get()
rhysd Mar 23, 2019
a785c80
Move description of float window behavior to function doc section
rhysd Mar 23, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix checking enough space for floating window
  • Loading branch information
@rhysd
rhysd committed Mar 14, 2019
commit 5ccf450118b7259b09362ed80b034757c14f4dff
7 changes: 4 additions & 3 deletions autoload/LanguageClient.vim
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -271,15 +271,15 @@ function! s:CloseFloatingHoverAfterCursorMove(win_id, opened) abort
endif
autocmd! plugin-LC-neovim-close-hover
let winnr = win_id2win(a:win_id)
if winnr == -1
if winnr == 0
return
endif
execute winnr . 'wincmd c'
endfunction

function! s:CloseFloatingHoverAfterEnterAnotherWin(win_id) abort
let winnr = win_id2win(a:win_id)
if winnr == -1
if winnr == 0
" Float window was already closed
autocmd! plugin-LC-neovim-close-hover
return
Expand Down Expand Up @@ -320,7 +320,8 @@ function! s:OpenHoverPreview(bufname, lines, filetype) abort

" Calculate anchor
" Prefer North, but if there is no space, fallback into South
if pos[1] + height <= &lines
let bottom_line = line('w0') + winheight() - 1
if pos[1] + height <= bottom_line
let vert = 'N'
let row = 1
else
Expand Down