mirror of
https://github.com/neovim/neovim.git
synced 2025-02-25 18:55:25 -06:00
vim-patch:9.0.1731: blockwise Visual highlight not working with virtual text (#24779)
Problem: blockwise Visual highlight not working with virtual text
Solution: Reset the correct variable at the end of virtual selection and
Check for double-width char inside virtual text.
closes: vim/vim#12606
6e940d9a1d
Need to remove area_active and use wlv.fromcol and wlv.tocol directly.
This commit is contained in:
zeertzjq
GitHub
parent
8d7a2a1aea
commit
81d5550d77
@ -106,7 +106,7 @@ typedef struct {
|
|||||||
int c_extra; ///< extra chars, all the same
|
int c_extra; ///< extra chars, all the same
|
||||||
int c_final; ///< final char, mandatory if set
|
int c_final; ///< final char, mandatory if set
|
||||||
|
|
||||||
bool extra_for_extmark;
|
bool extra_for_extmark; ///< n_extra set for inline virtual text
|
||||||
|
|
||||||
// saved "extra" items for when draw_state becomes WL_LINE (again)
|
// saved "extra" items for when draw_state becomes WL_LINE (again)
|
||||||
int saved_n_extra;
|
int saved_n_extra;
|
||||||
@ -1147,8 +1147,6 @@ int win_line(win_T *wp, linenr_T lnum, int startrow, int endrow, bool number_onl
|
|||||||
|
|
||||||
char buf_fold[FOLD_TEXT_LEN]; // Hold value returned by get_foldtext
|
char buf_fold[FOLD_TEXT_LEN]; // Hold value returned by get_foldtext
|
||||||
|
|
||||||
bool area_active = false;
|
|
||||||
|
|
||||||
// 'cursorlineopt' has "screenline" and cursor is in this line
|
// 'cursorlineopt' has "screenline" and cursor is in this line
|
||||||
bool cul_screenline = false;
|
bool cul_screenline = false;
|
||||||
// margin columns for the screen line, needed for when 'cursorlineopt'
|
// margin columns for the screen line, needed for when 'cursorlineopt'
|
||||||
@ -1786,32 +1784,14 @@ int win_line(win_T *wp, linenr_T lnum, int startrow, int endrow, bool number_onl
|
|||||||
int extmark_attr = 0;
|
int extmark_attr = 0;
|
||||||
if (wlv.draw_state == WL_LINE
|
if (wlv.draw_state == WL_LINE
|
||||||
&& (area_highlighting || spv->spv_has_spell || extra_check)) {
|
&& (area_highlighting || spv->spv_has_spell || extra_check)) {
|
||||||
// handle Visual or match highlighting in this line
|
|
||||||
if (wlv.vcol == wlv.fromcol
|
|
||||||
|| (wlv.vcol + 1 == wlv.fromcol
|
|
||||||
&& ((wlv.n_extra == 0 && utf_ptr2cells(ptr) > 1)
|
|
||||||
|| (wlv.n_extra > 0 && wlv.p_extra != NULL
|
|
||||||
&& utf_ptr2cells(wlv.p_extra) > 1)))
|
|
||||||
|| (vcol_prev == fromcol_prev
|
|
||||||
&& vcol_prev < wlv.vcol // not at margin
|
|
||||||
&& wlv.vcol < wlv.tocol)) {
|
|
||||||
area_attr = vi_attr; // start highlighting
|
|
||||||
if (area_highlighting) {
|
|
||||||
area_active = true;
|
|
||||||
}
|
|
||||||
} else if (area_attr != 0 && (wlv.vcol == wlv.tocol
|
|
||||||
|| (noinvcur && wlv.vcol == wp->w_virtcol))) {
|
|
||||||
area_attr = 0; // stop highlighting
|
|
||||||
area_active = false;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (wlv.n_extra == 0 || !wlv.extra_for_extmark) {
|
if (wlv.n_extra == 0 || !wlv.extra_for_extmark) {
|
||||||
wlv.reset_extra_attr = false;
|
wlv.reset_extra_attr = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (has_decor && wlv.n_extra == 0) {
|
if (has_decor && wlv.n_extra == 0) {
|
||||||
bool selected = (area_active || (area_highlighting && noinvcur
|
bool selected = (area_highlighting
|
||||||
&& wlv.vcol == wp->w_virtcol));
|
&& ((wlv.vcol >= wlv.fromcol && wlv.vcol < wlv.tocol)
|
||||||
|
|| (noinvcur && wlv.vcol == wp->w_virtcol)));
|
||||||
extmark_attr = decor_redraw_col(wp, (colnr_T)v, wlv.off, selected, &decor_state);
|
extmark_attr = decor_redraw_col(wp, (colnr_T)v, wlv.off, selected, &decor_state);
|
||||||
|
|
||||||
if (!has_fold && wp->w_buffer->b_virt_text_inline > 0) {
|
if (!has_fold && wp->w_buffer->b_virt_text_inline > 0) {
|
||||||
@ -1827,11 +1807,33 @@ int win_line(win_T *wp, linenr_T lnum, int startrow, int endrow, bool number_onl
|
|||||||
area_attr = 0;
|
area_attr = 0;
|
||||||
decor_attr = 0;
|
decor_attr = 0;
|
||||||
search_attr_from_match = false;
|
search_attr_from_match = false;
|
||||||
|
}
|
||||||
|
if (wlv.n_extra > 0) {
|
||||||
n_skip = 0;
|
n_skip = 0;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
int *area_attr_p
|
||||||
|
= wlv.extra_for_extmark && wlv.virt_inline_hl_mode <= kHlModeReplace
|
||||||
|
? &saved_area_attr : &area_attr;
|
||||||
|
|
||||||
|
// handle Visual or match highlighting in this line
|
||||||
|
if (wlv.vcol == wlv.fromcol
|
||||||
|
|| (wlv.vcol + 1 == wlv.fromcol
|
||||||
|
&& ((wlv.n_extra == 0 && utf_ptr2cells(ptr) > 1)
|
||||||
|
|| (wlv.n_extra > 0 && wlv.p_extra != NULL
|
||||||
|
&& utf_ptr2cells(wlv.p_extra) > 1)))
|
||||||
|
|| (vcol_prev == fromcol_prev
|
||||||
|
&& vcol_prev < wlv.vcol // not at margin
|
||||||
|
&& wlv.vcol < wlv.tocol)) {
|
||||||
|
*area_attr_p = vi_attr; // start highlighting
|
||||||
|
} else if (*area_attr_p != 0
|
||||||
|
&& (wlv.vcol == wlv.tocol
|
||||||
|
|| (noinvcur && wlv.vcol == wp->w_virtcol))) {
|
||||||
|
*area_attr_p = 0; // stop highlighting
|
||||||
|
}
|
||||||
|
|
||||||
if (!has_fold && wlv.n_extra == 0) {
|
if (!has_fold && wlv.n_extra == 0) {
|
||||||
// Check for start/end of 'hlsearch' and other matches.
|
// Check for start/end of 'hlsearch' and other matches.
|
||||||
// After end, check for start/end of next match.
|
// After end, check for start/end of next match.
|
||||||
|
|||||||
@ -2146,7 +2146,7 @@ bbbbbbb]])
|
|||||||
]]}
|
]]}
|
||||||
end)
|
end)
|
||||||
|
|
||||||
it('visual select highlight is correct', function()
|
it('Visual select highlight is correct', function()
|
||||||
insert('foo foo foo bar\nfoo foo foo bar')
|
insert('foo foo foo bar\nfoo foo foo bar')
|
||||||
feed('gg0')
|
feed('gg0')
|
||||||
meths.buf_set_extmark(0, ns, 0, 8, { virt_text = { { 'AAA', 'Special' } }, virt_text_pos = 'inline' })
|
meths.buf_set_extmark(0, ns, 0, 8, { virt_text = { { 'AAA', 'Special' } }, virt_text_pos = 'inline' })
|
||||||
@ -2803,6 +2803,152 @@ bbbbbbb]])
|
|||||||
|
|
|
|
||||||
]]}
|
]]}
|
||||||
end)
|
end)
|
||||||
|
|
||||||
|
it('blockwise Visual highlight with double-width virtual text (replace)', function()
|
||||||
|
screen:try_resize(60, 6)
|
||||||
|
insert('123456789\n123456789\n123456789')
|
||||||
|
meths.buf_set_extmark(0, ns, 1, 1, {
|
||||||
|
virt_text = { { '-口-', 'Special' } },
|
||||||
|
virt_text_pos = 'inline',
|
||||||
|
hl_mode = 'replace',
|
||||||
|
})
|
||||||
|
feed('gg0')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
^123456789 |
|
||||||
|
1{10:-口-}23456789 |
|
||||||
|
123456789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
|
|
||||||
|
]]}
|
||||||
|
feed('<C-V>2jl')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
{7:12}3456789 |
|
||||||
|
{7:1}{10:-口-}23456789 |
|
||||||
|
{7:1}^23456789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
{7:123}456789 |
|
||||||
|
{7:1}{10:-口-}23456789 |
|
||||||
|
{7:12}^3456789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('4l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
{7:1234567}89 |
|
||||||
|
{7:1}{10:-口-}{7:23}456789 |
|
||||||
|
{7:123456}^789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('Ol')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
1{7:234567}89 |
|
||||||
|
1{10:-口-}{7:23}456789 |
|
||||||
|
1^2{7:34567}89 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
12{7:34567}89 |
|
||||||
|
1{10:-口-}{7:23}456789 |
|
||||||
|
12^3{7:4567}89 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
123{7:4567}89 |
|
||||||
|
1{10:-口-}{7:23}456789 |
|
||||||
|
123^4{7:567}89 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
end)
|
||||||
|
|
||||||
|
it('blockwise Visual highlight with double-width virtual text (combine)', function()
|
||||||
|
screen:try_resize(60, 6)
|
||||||
|
insert('123456789\n123456789\n123456789')
|
||||||
|
meths.buf_set_extmark(0, ns, 1, 1, {
|
||||||
|
virt_text = { { '-口-', 'Special' } },
|
||||||
|
virt_text_pos = 'inline',
|
||||||
|
hl_mode = 'combine',
|
||||||
|
})
|
||||||
|
feed('gg0')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
^123456789 |
|
||||||
|
1{10:-口-}23456789 |
|
||||||
|
123456789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
|
|
||||||
|
]]}
|
||||||
|
feed('<C-V>2jl')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
{7:12}3456789 |
|
||||||
|
{7:1}{20:-}{10:口-}23456789 |
|
||||||
|
{7:1}^23456789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
{7:123}456789 |
|
||||||
|
{7:1}{20:-口}{10:-}23456789 |
|
||||||
|
{7:12}^3456789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('4l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
{7:1234567}89 |
|
||||||
|
{7:1}{20:-口-}{7:23}456789 |
|
||||||
|
{7:123456}^789 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('Ol')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
1{7:234567}89 |
|
||||||
|
1{20:-口-}{7:23}456789 |
|
||||||
|
1^2{7:34567}89 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
12{7:34567}89 |
|
||||||
|
1{10:-}{20:口-}{7:23}456789 |
|
||||||
|
12^3{7:4567}89 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
feed('l')
|
||||||
|
screen:expect{grid=[[
|
||||||
|
123{7:4567}89 |
|
||||||
|
1{10:-}{20:口-}{7:23}456789 |
|
||||||
|
123^4{7:567}89 |
|
||||||
|
{1:~ }|
|
||||||
|
{1:~ }|
|
||||||
|
{8:-- VISUAL BLOCK --} |
|
||||||
|
]]}
|
||||||
|
end)
|
||||||
end)
|
end)
|
||||||
|
|
||||||
describe('decorations: virtual lines', function()
|
describe('decorations: virtual lines', function()
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user