# HG changeset patch # User Christian Brabandt # Date 1693498507 -7200 # Node ID 4be1ffa13b5603a667d9d06f3cecd09039c05503 # Parent 65b122a95cbad989d3c08114617fe450f9dd8ed4 patch 9.0.1828: cursor wrong with virt text before double-width char Commit: https://github.com/vim/vim/commit/ac2d8815ae7a93c54b07cba76475cfb3f26a3ac6 Author: zeertzjq Date: Thu Aug 31 18:07:30 2023 +0200 patch 9.0.1828: cursor wrong with virt text before double-width char Problem: Wrong cursor position with virtual text before double-width char at window edge. Solution: Check for double-width char before adding virtual text size. closes: #12977 Signed-off-by: Christian Brabandt Co-authored-by: zeertzjq diff --git a/src/charset.c b/src/charset.c --- a/src/charset.c +++ b/src/charset.c @@ -1159,6 +1159,9 @@ win_lbr_chartabsize( size = win_chartabsize(wp, s, vcol); if (*s == NUL && !has_lcs_eol) size = 0; // NUL is not displayed +# ifdef FEAT_LINEBREAK + int is_doublewidth = has_mbyte && size == 2 && MB_BYTE2LEN(*s) > 1; +# endif # ifdef FEAT_PROP_POPUP if (cts->cts_has_prop_with_text) @@ -1242,8 +1245,7 @@ win_lbr_chartabsize( # endif # ifdef FEAT_LINEBREAK - if (has_mbyte && size == 2 && MB_BYTE2LEN(*s) > 1 - && wp->w_p_wrap && in_win_border(wp, vcol)) + if (is_doublewidth && wp->w_p_wrap && in_win_border(wp, vcol + size - 2)) { ++size; // Count the ">" in the last column. mb_added = 1; diff --git a/src/testdir/dumps/Test_prop_inserts_text_before_double_width_wrap_1.dump b/src/testdir/dumps/Test_prop_inserts_text_before_double_width_wrap_1.dump new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_before_double_width_wrap_1.dump @@ -0,0 +1,3 @@ +|a+0&#ffffff0@39|b+0#e000e06&@8|>+0#4040ff13& +|口*0#0000000&|1+&|2|3|4>5| @42 +@32|1|,|4|8|-|5|7| @6|A|l@1| diff --git a/src/testdir/test_textprop.vim b/src/testdir/test_textprop.vim --- a/src/testdir/test_textprop.vim +++ b/src/testdir/test_textprop.vim @@ -2810,6 +2810,22 @@ func Test_prop_inserts_text_before_lineb call StopVimInTerminal(buf) endfunc +func Test_prop_inserts_text_before_double_width_wrap() + CheckRunVimInTerminal + + let lines =<< trim END + call setline(1, repeat('a', 40) .. '口' .. '12345') + call prop_type_add('theprop', #{highlight: 'Special'}) + call prop_add(1, 41, #{type: 'theprop', text: repeat('b', 9)}) + normal! $ + END + call writefile(lines, 'XscriptPropsBeforeDoubleWidthWrap', 'D') + let buf = RunVimInTerminal('-S XscriptPropsBeforeDoubleWidthWrap', #{rows: 3, cols: 50}) + call VerifyScreenDump(buf, 'Test_prop_inserts_text_before_double_width_wrap_1', {}) + + call StopVimInTerminal(buf) +endfunc + func Test_prop_inserts_text_lcs_extends() CheckRunVimInTerminal diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -700,6 +700,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1828, +/**/ 1827, /**/ 1826,