diff options
author | Bram Moolenaar <Bram@vim.org> | 2019-08-04 21:12:15 +0200 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2019-08-04 21:12:15 +0200 |
commit | 5d458a7b3d1819d51bff6f3682b81bf943c7e9d0 (patch) | |
tree | e624fd72ded61881e9f7a82d2967836b8b48c965 | |
parent | 2debf1c16b93f8693a785f675320d9e949c96a97 (diff) |
patch 8.1.1814: a long title in a popup window overflowsv8.1.1814
Problem: A long title in a popup window overflows.
Solution: Truncate the title. (closes #4770)
-rw-r--r-- | src/popupwin.c | 8 | ||||
-rw-r--r-- | src/testdir/dumps/Test_popupwin_longtitle_1.dump | 10 | ||||
-rw-r--r-- | src/testdir/dumps/Test_popupwin_longtitle_2.dump | 10 | ||||
-rw-r--r-- | src/testdir/test_popupwin.vim | 10 | ||||
-rw-r--r-- | src/version.c | 2 |
5 files changed, 38 insertions, 2 deletions
diff --git a/src/popupwin.c b/src/popupwin.c index ca75181426..9d801ea4bc 100644 --- a/src/popupwin.c +++ b/src/popupwin.c @@ -2969,8 +2969,14 @@ update_popups(void (*win_update)(win_T *wp)) // Title goes on top of border or padding. if (wp->w_popup_title != NULL) - screen_puts(wp->w_popup_title, wp->w_winrow, wp->w_wincol + 1, + { + int len = (int)STRLEN(wp->w_popup_title) + 1; + char_u *title = alloc(len); + + trunc_string(wp->w_popup_title, title, total_width - 2, len); + screen_puts(title, wp->w_winrow, wp->w_wincol + 1, wp->w_popup_border[0] > 0 ? border_attr[0] : popup_attr); + } // Compute scrollbar thumb position and size. if (wp->w_has_scrollbar) diff --git a/src/testdir/dumps/Test_popupwin_longtitle_1.dump b/src/testdir/dumps/Test_popupwin_longtitle_1.dump new file mode 100644 index 0000000000..92af8b2594 --- /dev/null +++ b/src/testdir/dumps/Test_popupwin_longtitle_1.dump @@ -0,0 +1,10 @@ +>1+0&#ffffff0| @73 +|2| @73 +|3| @73 +|4| @25| +0#0000001#ffd7ff255|a| |v|e|r|y| |.@2|g| |t|o| |f|i|t| | +0#0000000#ffffff0@27 +|5| @25|o+0#0000001#ffd7ff255|n|e| @16| +0#0000000#ffffff0@27 +|6| @25|t+0#0000001#ffd7ff255|w|o| @16| +0#0000000#ffffff0@27 +|7| @25|a+0#0000001#ffd7ff255|n|o|t|h|e|r| @12| +0#0000000#ffffff0@27 +|8| @73 +|9| @73 +|:| @55|1|,|1| @10|T|o|p| diff --git a/src/testdir/dumps/Test_popupwin_longtitle_2.dump b/src/testdir/dumps/Test_popupwin_longtitle_2.dump new file mode 100644 index 0000000000..079fbc387c --- /dev/null +++ b/src/testdir/dumps/Test_popupwin_longtitle_2.dump @@ -0,0 +1,10 @@ +>1+0&#ffffff0| @73 +|2| @73 +|3| @24|╔+0#0000001#ffd7ff255|a| |v|e|r|y| |l|.@2|n|g| |t|o| |f|i|t|╗| +0#0000000#ffffff0@26 +|4| @24|║+0#0000001#ffd7ff255|o|n|e| @16|║| +0#0000000#ffffff0@26 +|5| @24|║+0#0000001#ffd7ff255|t|w|o| @16|║| +0#0000000#ffffff0@26 +|6| @24|║+0#0000001#ffd7ff255|a|n|o|t|h|e|r| @12|║| +0#0000000#ffffff0@26 +|7| @24|╚+0#0000001#ffd7ff255|═@19|╝| +0#0000000#ffffff0@26 +|8| @73 +|9| @73 +|:| @55|1|,|1| @10|T|o|p| diff --git a/src/testdir/test_popupwin.vim b/src/testdir/test_popupwin.vim index 8a5892670e..c1bae367b8 100644 --- a/src/testdir/test_popupwin.vim +++ b/src/testdir/test_popupwin.vim @@ -1282,12 +1282,20 @@ func Test_popup_title() " put the title on. let lines =<< trim END call setline(1, range(1, 20)) - call popup_create(['one', 'two', 'another'], #{title: 'Title String'}) + let winid = popup_create(['one', 'two', 'another'], #{title: 'Title String'}) END call writefile(lines, 'XtestPopupTitle') let buf = RunVimInTerminal('-S XtestPopupTitle', #{rows: 10}) call VerifyScreenDump(buf, 'Test_popupwin_title', {}) + call term_sendkeys(buf, ":call popup_setoptions(winid, #{maxwidth: 20, title: 'a very long title that is not going to fit'})\<CR>") + call term_sendkeys(buf, ":\<CR>") + call VerifyScreenDump(buf, 'Test_popupwin_longtitle_1', {}) + + call term_sendkeys(buf, ":call popup_setoptions(winid, #{border: []})\<CR>") + call term_sendkeys(buf, ":\<CR>") + call VerifyScreenDump(buf, 'Test_popupwin_longtitle_2', {}) + " clean up call StopVimInTerminal(buf) call delete('XtestPopupTitle') diff --git a/src/version.c b/src/version.c index 7c9ba68ad5..b8479b15c6 100644 --- a/src/version.c +++ b/src/version.c @@ -774,6 +774,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1814, +/**/ 1813, /**/ 1812, |