Hi Linwei, 2018-2-22(Thu) 14:59:27 UTC+9 Linwei: > Expected position (v8.0.1489): > > > > In v8.0.1489, the popupmenu is adaptive to the cursor's position. The left > edge of > > popupmenu corresponds to the cursor. > > Actuall position (v8.0.1529): > > > > But in v8.0.1529, popupmenu's left edge is not in the same column of cursor. > I need move my eyes away from the cursor position to the left side to see the > completion candidates, which is not convenient. > > So, can popupmenu show itself in the old way ?
Thanks for reporting. Can you confirm an attached patch? Bram> Sorry for not having test. -- Best regards, Hirohito Higashi (h_east) -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
diff --git a/src/popupmnu.c b/src/popupmnu.c index dfdcca072..6ac121d91 100644 --- a/src/popupmnu.c +++ b/src/popupmnu.c @@ -253,7 +253,7 @@ pum_display( /* align right pum edge with "col" */ #ifdef FEAT_RIGHTLEFT if (curwin->w_p_rl - && col < max_width + pum_scrollbar + 1) + && W_ENDCOL(curwin) < max_width + pum_scrollbar + 1) { pum_col = col + max_width + pum_scrollbar + 1; if (pum_col >= Columns) @@ -262,7 +262,7 @@ pum_display( else if (!curwin->w_p_rl) #endif { - if (col > Columns - max_width - pum_scrollbar) + if (curwin->w_wincol > Columns - max_width - pum_scrollbar) { pum_col = Columns - max_width - pum_scrollbar; if (pum_col < 0)