commit a37cb55da6ba44c5e85a22b03a91e1b663aceddf
author Bram Moolenaar <Bram@vim.org> Sat Nov 16 20:03:31 2019 +0100
committer Bram Moolenaar <Bram@vim.org> Sat Nov 16 20:03:31 2019 +0100
tree d849289e4512042e6d1a86ebba9ebdbd8d033d0d
parent f36a2c7e60d86b1a1733a8f51ed72da9c6f76eb8

patch 8.1.2307: positioning popup doesn't work for buffer-local textprop

Problem:    Positioning popup doesn't work for buffer-local textprop.
Solution:   Make it work. (closes #5225)

src/popupwin.c

diff --git a/src/popupwin.c b/src/popupwin.c
index 48c017f..efcc395 100644
--- a/src/popupwin.c
+++ b/src/popupwin.c
@@ -452,24 +452,22 @@
 	wp->w_popup_prop_type = 0;
 	if (*str != NUL)
 	{
-	    nr = find_prop_type_id(str, wp->w_buffer);
+	    wp->w_popup_prop_win = curwin;
+	    di = dict_find(d, (char_u *)"textpropwin", -1);
+	    if (di != NULL)
+	    {
+		wp->w_popup_prop_win = find_win_by_nr_or_id(&di->di_tv);
+		if (!win_valid(wp->w_popup_prop_win))
+		    wp->w_popup_prop_win = curwin;
+	    }
+
+	    nr = find_prop_type_id(str, wp->w_popup_prop_win->w_buffer);
 	    if (nr <= 0)
 		nr = find_prop_type_id(str, NULL);
 	    if (nr <= 0)
 		semsg(_(e_invarg2), str);
 	    else
-	    {
 		wp->w_popup_prop_type = nr;
-		wp->w_popup_prop_win = curwin;
-
-		di = dict_find(d, (char_u *)"textpropwin", -1);
-		if (di != NULL)
-		{
-		    wp->w_popup_prop_win = find_win_by_nr_or_id(&di->di_tv);
-		    if (win_valid(wp->w_popup_prop_win))
-			wp->w_popup_prop_win = curwin;
-		}
-	    }
 	}
     }
 

src/testdir/test_popupwin_textprop.vim

diff --git a/src/testdir/test_popupwin_textprop.vim b/src/testdir/test_popupwin_textprop.vim
index 8a68f15..b112755 100644
--- a/src/testdir/test_popupwin_textprop.vim
+++ b/src/testdir/test_popupwin_textprop.vim
@@ -13,10 +13,11 @@
 	50
 	normal zz
 	set scrolloff=0
-	call prop_type_add('popupMarker', #{highlight: 'DiffAdd'})
+	call prop_type_add('popupMarker', #{highlight: 'DiffAdd', bufnr: bufnr('%')})
 	call prop_add(50, 11, #{
 		\ length: 7,
 		\ type: 'popupMarker',
+		\ bufnr: bufnr('%'),
 		\ })
 	let winid = popup_create('the text', #{
 	      \ pos: 'botleft', 
@@ -68,6 +69,7 @@
 	let winid = popup_create('bottom left', #{
 	      \ pos: 'botleft', 
 	      \ textprop: 'popupMarker',
+	      \ textpropwin: win_getid(),
 	      \ padding: [0,1,0,1],
 	      \ })
 	let winid = popup_create('bottom right', #{

src/version.c

diff --git a/src/version.c b/src/version.c
index dc6caed..a682517 100644
--- a/src/version.c
+++ b/src/version.c
@@ -742,6 +742,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    2307,
+/**/
     2306,
 /**/
     2305,