From 749fa0af85232be1d44b77a09161f71cdbace62c Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Sat, 3 Aug 2019 16:18:07 +0200 Subject: patch 8.1.1799: cannot avoid mapping for a popup window Problem: Cannot avoid mapping for a popup window. Solution: Add the "mapping" property, default TRUE. --- src/getchar.c | 3 +++ src/popupwin.c | 31 ++++++++++++++++++++++++++++++- src/proto/popupwin.pro | 1 + src/testdir/test_popupwin.vim | 7 +++++++ src/version.c | 2 ++ src/vim.h | 1 + 6 files changed, 44 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/getchar.c b/src/getchar.c index fe1cc7e2ef..4a3587c3f2 100644 --- a/src/getchar.c +++ b/src/getchar.c @@ -1574,6 +1574,9 @@ vgetc(void) if (mod_mask #if defined(FEAT_XIM) && defined(FEAT_GUI_GTK) || im_is_preediting() +#endif +#if defined(FEAT_TEXT_PROP) + || popup_no_mapping() #endif ) { diff --git a/src/popupwin.c b/src/popupwin.c index 4ca6d9fdf1..28ad48cfc8 100644 --- a/src/popupwin.c +++ b/src/popupwin.c @@ -800,6 +800,15 @@ apply_general_options(win_T *wp, dict_T *dict) set_callback(&wp->w_filter_cb, &callback); } } + di = dict_find(dict, (char_u *)"mapping", -1); + if (di != NULL) + { + nr = dict_get_number(dict, (char_u *)"mapping"); + if (nr) + wp->w_popup_flags |= POPF_MAPPING; + else + wp->w_popup_flags &= ~POPF_MAPPING; + } di = dict_find(dict, (char_u *)"callback", -1); if (di != NULL) @@ -1413,7 +1422,7 @@ popup_create(typval_T *argvars, typval_T *rettv, create_type_T type) if (rettv != NULL) rettv->vval.v_number = wp->w_id; wp->w_popup_pos = POPPOS_TOPLEFT; - wp->w_popup_flags = POPF_IS_POPUP; + wp->w_popup_flags = POPF_IS_POPUP | POPF_MAPPING; if (buf != NULL) { @@ -1561,6 +1570,7 @@ popup_create(typval_T *argvars, typval_T *rettv, create_type_T type) wp->w_popup_pos = POPPOS_CENTER; wp->w_zindex = POPUPWIN_DIALOG_ZINDEX; wp->w_popup_flags |= POPF_DRAG; + wp->w_popup_flags &= ~POPF_MAPPING; for (i = 0; i < 4; ++i) { wp->w_popup_border[i] = 1; @@ -2501,6 +2511,25 @@ popup_do_filter(int c) return res; } +/* + * Return TRUE if there is a popup visible with a filter callback and the + * "mapping" property off. + */ + int +popup_no_mapping(void) +{ + int round; + win_T *wp; + + for (round = 1; round <= 2; ++round) + for (wp = round == 1 ? first_popupwin : curtab->tp_first_popupwin; + wp != NULL; wp = wp->w_next) + if (wp->w_filter_cb.cb_name != NULL + && (wp->w_popup_flags & (POPF_HIDDEN | POPF_MAPPING)) == 0) + return TRUE; + return FALSE; +} + /* * Called when the cursor moved: check if any popup needs to be closed if the * cursor moved far enough. diff --git a/src/proto/popupwin.pro b/src/proto/popupwin.pro index b964fc0dd0..f1d0d2b523 100644 --- a/src/proto/popupwin.pro +++ b/src/proto/popupwin.pro @@ -38,6 +38,7 @@ int error_if_popup_window(void); void popup_reset_handled(void); win_T *find_next_popup(int lowest); int popup_do_filter(int c); +int popup_no_mapping(void); void popup_check_cursor_pos(void); void may_update_popup_mask(int type); void update_popups(void (*win_update)(win_T *wp)); diff --git a/src/testdir/test_popupwin.vim b/src/testdir/test_popupwin.vim index d4b98aee7b..e5d16314bc 100644 --- a/src/testdir/test_popupwin.vim +++ b/src/testdir/test_popupwin.vim @@ -657,6 +657,7 @@ func Test_popup_invalid_arguments() call assert_fails('call popup_create("text", #{mask: ["asdf"]})', 'E475:') call popup_clear() call assert_fails('call popup_create("text", #{mask: test_null_list()})', 'E475:') + call assert_fails('call popup_create("text", #{mapping: []})', 'E745:') call popup_clear() endfunc @@ -1203,6 +1204,8 @@ func Test_popup_menu() let s:cb_winid = a:id let s:cb_res = a:res endfunc + " mapping won't be used in popup + map j k let winid = ShowMenu(" ", 1) let winid = ShowMenu("j \", 2) @@ -1215,6 +1218,7 @@ func Test_popup_menu() let winid = ShowMenu("\", -1) delfunc QuitCallback + unmap j endfunc func Test_popup_menu_screenshot() @@ -2194,6 +2198,9 @@ func Test_previewpopup() call term_sendkeys(buf, "/another\\}") call VerifyScreenDump(buf, 'Test_popupwin_previewpopup_4', {}) + call term_sendkeys(buf, ":cd ..\:\") + call VerifyScreenDump(buf, 'Test_popupwin_previewpopup_5', {}) + call StopVimInTerminal(buf) call delete('Xtags') call delete('Xtagfile') diff --git a/src/version.c b/src/version.c index ec67fe4060..b2c4416ae3 100644 --- a/src/version.c +++ b/src/version.c @@ -773,6 +773,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 1799, /**/ 1798, /**/ diff --git a/src/vim.h b/src/vim.h index 02b4c03bc6..70fcf3a12c 100644 --- a/src/vim.h +++ b/src/vim.h @@ -621,6 +621,7 @@ extern int (*dyn_libintl_wputenv)(const wchar_t *envstring); #define POPF_ON_CMDLINE 0x10 // popup overlaps command line #define POPF_DRAG 0x20 // popup can be moved by dragging #define POPF_RESIZE 0x40 // popup can be resized by dragging +#define POPF_MAPPING 0x80 // mapping keys #ifdef FEAT_TEXT_PROP # define WIN_IS_POPUP(wp) ((wp)->w_popup_flags != 0) -- cgit v1.2.3