Skip to content

Commit d6bcff4

Browse files
committed
patch 8.1.1712: signs in number column cause text to be misaligned
Problem: Signs in number column cause text to be misaligned. Solution: Improve alignment. (Yasuhiro Matsumoto, closes #4694)
1 parent acf7544 commit d6bcff4

3 files changed

Lines changed: 16 additions & 2 deletions

File tree

src/screen.c

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3120,8 +3120,13 @@ get_sign_display_info(
31203120
{
31213121
if (nrcol)
31223122
{
3123-
sprintf((char *)extra, "%*s ", number_width(wp),
3124-
*pp_extra);
3123+
int n, width = number_width(wp) - 2;
3124+
3125+
for (n = 0; n < width; n++)
3126+
extra[n] = ' ';
3127+
extra[n] = 0;
3128+
STRCAT(extra, *pp_extra);
3129+
STRCAT(extra, " ");
31253130
*pp_extra = extra;
31263131
}
31273132
*c_extrap = NUL;

src/testdir/test_signs.vim

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1766,6 +1766,7 @@ func Test_sign_numcol()
17661766
set number
17671767
set signcolumn=number
17681768
sign define sign1 text==>
1769+
sign define sign2 text=
17691770
sign place 10 line=1 name=sign1
17701771
redraw!
17711772
call assert_equal("=> 01234", s:ScreenLine(1, 1, 8))
@@ -1846,6 +1847,12 @@ func Test_sign_numcol()
18461847
redraw!
18471848
call assert_equal("=> 01234", s:ScreenLine(1, 1, 8))
18481849
call assert_equal(" 2 abcde", s:ScreenLine(2, 1, 8))
1850+
" Add sign with multi-byte text
1851+
set numberwidth=4
1852+
sign place 40 line=2 name=sign2
1853+
redraw!
1854+
call assert_equal(" => 01234", s:ScreenLine(1, 1, 9))
1855+
call assert_equal(" V abcde", s:ScreenLine(2, 1, 9))
18491856

18501857
sign unplace * group=*
18511858
sign undefine sign1

src/version.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -777,6 +777,8 @@ static char *(features[]) =
777777

778778
static int included_patches[] =
779779
{ /* Add new patch number below this line */
780+
/**/
781+
1712,
780782
/**/
781783
1711,
782784
/**/

0 commit comments

Comments
 (0)