Use Vim8.1 popup feature

[rhysd]

https://github.com/vim/vim/blob/master/runtime/doc/popup.txt

It seems about to be being implemented. If it works and sufficient for this plugin, using it would be better for Vim users.

[airblade]

As far as I understand a popup window cannot be focused so it wouldn't be possible to use the mappings etc.

[rhysd]

Yeah, the point is a blocker. Currently I don't have good idea to workaround that.

[airblade]

I think if the user wants to use popups to see the blame info, it's useful even if they can't do anything with it. I suppose you can make the popup as big as the blame info, up to the size of the window, to minimise scrolling.

[rhysd]

Current implementation determines size of float window following contents. However, sometimes commit body is too big. And if user wants to see diff of the commit, it would be not fit to window (diff is usually bigger).

Another point is local key maps in the float window. This plugin provides some mappings such as o (show older commit). I also don't have good idea to provide the mappings without focusing (entering) a popup. Neovim's float window allows to move cursor into popup, so we can define window-local mappings. But that seems not available in Vim.

[blayz3r]

vim/vim#4577
mhinz/vim-signify#302

[tracyone]

:help previewpopup

[blayz3r]

bump

[tsuyoshicho]

I think operation likely that.
bfrg/vim-qf-preview: Vim plugin to preview current quickfix item in a popup window (Vim-only)

(skip into preview; direct operation.)

[blayz3r]

nmap <silent><Leader>g :call setbufvar(winbufnr(popup_atcursor(split(system("git log -n 1 -L " . line(".") . ",+1:" . expand("%:p")), "\n"), { "padding": [1,1,1,1], "pos": "botleft", "wrap": 0 })), "&filetype", "git")<CR>

co -> https://www.reddit.com/user/sheerun/

[rhysd]

All right, I'll make time to try to implement this feature this week.

[blayz3r]

Discussion here:
vim/vim#5639 (comment)