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)
