Mercurial > vim
changeset 33017:7d0357f70cf8 v9.0.1800
patch 9.0.1800: Cursor position still wrong with 'showbreak' and virtual text
Commit: https://github.com/vim/vim/commit/6a3897232aecd3e8b9e8b23955e55c1993e5baec
Author: zeertzjq <zeertzjq@outlook.com>
Date: Sun Aug 27 19:04:14 2023 +0200
patch 9.0.1800: Cursor position still wrong with 'showbreak' and virtual text
Problem: Cursor position still wrong with 'showbreak' and virtual text
after last character or 'listchars' "eol".
Solution: Remove unnecessary w_wcol adjustment in curs_columns(). Also
fix first char of virtual text not shown at the start of a screen
line.
closes: #12478
closes: #12532
closes: #12904
Signed-off-by: Christian Brabandt <cb@256bit.org>
Co-authored-by: zeertzjq <zeertzjq@outlook.com>
| author | Christian Brabandt <cb@256bit.org> |
|---|---|
| date | Sun, 27 Aug 2023 19:15:05 +0200 |
| parents | 0af63fa799bf |
| children | f0ac5bb79f9f |
| files | src/charset.c src/drawline.c src/move.c src/testdir/dumps/Test_cursor_position_with_showbreak_3.dump src/testdir/dumps/Test_prop_inserts_text_lcs_extends_1.dump src/testdir/dumps/Test_prop_inserts_text_lcs_extends_2.dump src/testdir/dumps/Test_prop_inserts_text_lcs_extends_3.dump src/testdir/dumps/Test_prop_inserts_text_lcs_extends_4.dump src/testdir/dumps/Test_prop_inserts_text_lcs_extends_5.dump src/testdir/dumps/Test_prop_inserts_text_showbreak_23.dump src/testdir/dumps/Test_prop_inserts_text_showbreak_24.dump src/testdir/test_breakindent.vim src/testdir/test_textprop.vim src/version.c |
| diffstat | 14 files changed, 80 insertions(+), 18 deletions(-) [+] |
line wrap: on
line diff
--- a/src/charset.c +++ b/src/charset.c @@ -1358,13 +1358,13 @@ win_lbr_chartabsize( head_mid += get_breakindent_win(wp, line); if (head_mid > 0 && wcol + size > wp->w_width) { - // calculate effective window width + // Calculate effective window width. int prev_rem = wp->w_width - wcol; int width = width2 - head_mid; if (width <= 0) width = 1; - // divide "size - prev_width" by "width", rounding up + // Divide "size - prev_rem" by "width", rounding up. int cnt = (size - prev_rem + width - 1) / width; added += cnt * head_mid;
--- a/src/drawline.c +++ b/src/drawline.c @@ -3733,7 +3733,11 @@ win_line( && (*ptr != NUL || lcs_eol_one > 0 || (wlv.n_extra > 0 && (wlv.c_extra != NUL - || *wlv.p_extra != NUL)))) + || *wlv.p_extra != NUL)) +#ifdef FEAT_PROP_POPUP + || text_prop_next < text_prop_count +#endif + )) { c = wp->w_lcs_chars.ext; wlv.char_attr = hl_combine_attr(wlv.win_attr, HL_ATTR(HLF_AT)); @@ -4014,6 +4018,7 @@ win_line( #endif #ifdef FEAT_PROP_POPUP || text_prop_above || text_prop_follows + || text_prop_next < text_prop_count #endif || (wp->w_p_list && wp->w_lcs_chars.eol != NUL && wlv.p_extra != at_end_str)
--- a/src/move.c +++ b/src/move.c @@ -1205,17 +1205,6 @@ curs_columns( n = (curwin->w_wcol - curwin->w_width) / width2 + 1; curwin->w_wcol -= n * width2; curwin->w_wrow += n; - -#ifdef FEAT_LINEBREAK - // When cursor wraps to first char of next line in Insert - // mode, the 'showbreak' string isn't shown, backup to first - // column - char_u *sbr = get_showbreak_value(curwin); - if (*sbr && *ml_get_cursor() == NUL - && curwin->w_wcol - == (curwin->w_width - width2) + vim_strsize(sbr)) - curwin->w_wcol = 0; -#endif } }
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_cursor_position_with_showbreak_3.dump @@ -0,0 +1,6 @@ +| +0#0000e05#a8a8a8255@1|x+0#0000000#ffffff0@71|X +| +0#0000e05#a8a8a8255@1|++0#4040ff13#ffffff0>$| +0#0000000&@70 +| +0#0000e05#a8a8a8255@1|s+0#0000000#ffffff0|e|c|o|n|d| |l|i|n|e|$+0#4040ff13&| +0#0000000&@60 +|~+0#4040ff13&| @73 +|~| @73 +|-+2#0000000&@1| |I|N|S|E|R|T| |-@1| +0&&@44|1|,|7|4| @9|A|l@1|
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_lcs_extends_1.dump @@ -0,0 +1,3 @@ +|a+0&#ffffff0@19>a@28|!+0#4040ff13& +|~| @48 +| +0#0000000&@31|1|,|2|1| @9|A|l@1|
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_lcs_extends_2.dump @@ -0,0 +1,3 @@ +|a+0&#ffffff0@18>a@29|!+0#4040ff13& +|~| @48 +| +0#0000000&@31|1|,|2|1| @9|A|l@1|
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_lcs_extends_3.dump @@ -0,0 +1,3 @@ +|a+0&#ffffff0@17>a@30|!+0#4040ff13& +|~| @48 +| +0#0000000&@31|1|,|2|1| @9|A|l@1|
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_lcs_extends_4.dump @@ -0,0 +1,3 @@ +|a+0&#ffffff0@16>a@30|b+0#e000e06&|!+0#4040ff13& +|~| @48 +| +0#0000000&@31|1|,|2|1| @9|A|l@1|
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_lcs_extends_5.dump @@ -0,0 +1,3 @@ +|a+0&#ffffff0@15>a@30|b+0#e000e06&@2 +|~+0#4040ff13&| @48 +| +0#0000000&@31|1|,|2|1| @9|A|l@1|
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_showbreak_23.dump @@ -0,0 +1,6 @@ +| +0&#ffffff0@1|1| |a@25 +| @5|++0#4040ff13&>1+0#e000e06&|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2 +| +0#0000000&@5|++0#4040ff13&|3+0#e000e06&|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1 +| +0#0000000&@5|++0#4040ff13&|2+0#e000e06&|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3 +|~+0#4040ff13&| @28 +|-+2#0000000&@1| |I|N|S|E|R|T| |-@1| +0&&@17
new file mode 100644 --- /dev/null +++ b/src/testdir/dumps/Test_prop_inserts_text_showbreak_24.dump @@ -0,0 +1,6 @@ +| +0&#ffffff0@1|1| |a@25 +| @3|++0#4040ff13&>1+0#e000e06&|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1 +| +0#0000000&@3|++0#4040ff13&|2+0#e000e06&|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2 +| +0#0000000&@3|++0#4040ff13&|3+0#e000e06&|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3|1|2|3| +0#0000000&@5 +|~+0#4040ff13&| @28 +|-+2#0000000&@1| |I|N|S|E|R|T| |-@1| +0&&@17
--- a/src/testdir/test_breakindent.vim +++ b/src/testdir/test_breakindent.vim @@ -898,7 +898,6 @@ func Test_cursor_position_with_showbreak vim9script &signcolumn = 'yes' &showbreak = '++' - &breakindent = true &breakindentopt = 'shift:2' var leftcol: number = win_getid()->getwininfo()->get(0, {})->get('textoff') repeat('x', &columns - leftcol - 1)->setline(1) @@ -912,9 +911,14 @@ func Test_cursor_position_with_showbreak " No line wraps, so changing 'showbreak' should lead to the same screen. call term_sendkeys(buf, "\<C-\>\<C-O>:setlocal showbreak=+\<CR>") call VerifyScreenDump(buf, 'Test_cursor_position_with_showbreak_1', {}) + " No line wraps, so setting 'breakindent' should lead to the same screen. + call term_sendkeys(buf, "\<C-\>\<C-O>:setlocal breakindent\<CR>") + call VerifyScreenDump(buf, 'Test_cursor_position_with_showbreak_1', {}) " The first line now wraps because of "eol" in 'listchars'. call term_sendkeys(buf, "\<C-\>\<C-O>:setlocal list\<CR>") call VerifyScreenDump(buf, 'Test_cursor_position_with_showbreak_2', {}) + call term_sendkeys(buf, "\<C-\>\<C-O>:setlocal nobreakindent\<CR>") + call VerifyScreenDump(buf, 'Test_cursor_position_with_showbreak_3', {}) call StopVimInTerminal(buf) endfunc
--- a/src/testdir/test_textprop.vim +++ b/src/testdir/test_textprop.vim @@ -2686,11 +2686,11 @@ func Run_test_prop_inserts_text_showbrea let lines =<< trim END highlight! link LineNr Normal + setlocal number showbreak=+ breakindent breakindentopt=shift:2 + setlocal scrolloff=0 smoothscroll call setline(1, repeat('a', 28)) call prop_type_add('theprop', #{highlight: 'Special'}) call prop_add(1, 28, #{type: 'theprop', text: repeat('123', 23), text_wrap: 'wrap'}) - setlocal number showbreak=+ breakindent breakindentopt=shift:2 - setlocal scrolloff=0 smoothscroll normal! $ END let lines = insert(lines, a:cmd) @@ -2740,6 +2740,10 @@ func Run_test_prop_inserts_text_showbrea call VerifyScreenDump(buf, 'Test_prop_inserts_text_showbreak_21', {}) call term_sendkeys(buf, "zbx") call VerifyScreenDump(buf, 'Test_prop_inserts_text_showbreak_22', {}) + call term_sendkeys(buf, "26ia\<Esc>a") + call VerifyScreenDump(buf, 'Test_prop_inserts_text_showbreak_23', {}) + call term_sendkeys(buf, "\<C-\>\<C-O>:setlocal breakindentopt=\<CR>") + call VerifyScreenDump(buf, 'Test_prop_inserts_text_showbreak_24', {}) call StopVimInTerminal(buf) endfunc @@ -2754,10 +2758,10 @@ func Test_prop_before_tab_skipcol() CheckRunVimInTerminal let lines =<< trim END + setlocal list listchars=tab:<-> scrolloff=0 smoothscroll call setline(1, repeat("\t", 4) .. 'a') call prop_type_add('theprop', #{highlight: 'Special'}) call prop_add(1, 4, #{type: 'theprop', text: repeat('12', 32), text_wrap: 'wrap'}) - setlocal list listchars=tab:<-> scrolloff=0 smoothscroll normal! $ END call writefile(lines, 'XscriptPropsBeforeTabSkipcol', 'D') @@ -2787,6 +2791,31 @@ func Test_prop_before_tab_skipcol() call StopVimInTerminal(buf) endfunc +func Test_prop_inserts_text_lcs_extends() + CheckRunVimInTerminal + + let lines =<< trim END + setlocal nowrap list listchars=extends:! + call setline(1, repeat('a', &columns + 1)) + call prop_type_add('theprop', #{highlight: 'Special'}) + call prop_add(1, &columns + 2, #{type: 'theprop', text: 'bbb'}) + END + call writefile(lines, 'XscriptPropsListExtends', 'D') + let buf = RunVimInTerminal('-S XscriptPropsListExtends', #{rows: 3, cols: 50}) + call term_sendkeys(buf, '20l') + call VerifyScreenDump(buf, 'Test_prop_inserts_text_lcs_extends_1', {}) + call term_sendkeys(buf, 'zl') + call VerifyScreenDump(buf, 'Test_prop_inserts_text_lcs_extends_2', {}) + call term_sendkeys(buf, 'zl') + call VerifyScreenDump(buf, 'Test_prop_inserts_text_lcs_extends_3', {}) + call term_sendkeys(buf, 'zl') + call VerifyScreenDump(buf, 'Test_prop_inserts_text_lcs_extends_4', {}) + call term_sendkeys(buf, 'zl') + call VerifyScreenDump(buf, 'Test_prop_inserts_text_lcs_extends_5', {}) + + call StopVimInTerminal(buf) +endfunc + func Test_prop_add_with_text_fails() call prop_type_add('failing', #{highlight: 'ErrorMsg'}) call assert_fails("call prop_add(1, 0, #{type: 'failing', text: 'X', end_lnum: 1})", 'E1305:')