patch 9.0.0469: cursor moves if cmdwin is closed when 'splitscroll' is off

Problem:    Cursor moves if cmdwin is closed when 'splitscroll' is off.
Solution:   Skip win_fix_cursor if called when cmdwin is open or closing.
            (Luuk van Baal, closes #11134)
This commit is contained in:
Luuk van Baal
2022-09-15 12:43:26 +01:00
committed by Bram Moolenaar
parent aeef1f7f6d
commit 3735f11050
4 changed files with 13 additions and 4 deletions

View File

@ -4405,7 +4405,6 @@ open_cmdwin(void)
int save_restart_edit = restart_edit; int save_restart_edit = restart_edit;
int save_State = State; int save_State = State;
int save_exmode = exmode_active; int save_exmode = exmode_active;
int save_p_spsc;
#ifdef FEAT_RIGHTLEFT #ifdef FEAT_RIGHTLEFT
int save_cmdmsg_rl = cmdmsg_rl; int save_cmdmsg_rl = cmdmsg_rl;
#endif #endif
@ -4645,10 +4644,8 @@ open_cmdwin(void)
wp = curwin; wp = curwin;
set_bufref(&bufref, curbuf); set_bufref(&bufref, curbuf);
save_p_spsc = p_spsc; skip_win_fix_cursor = TRUE;
p_spsc = TRUE;
win_goto(old_curwin); win_goto(old_curwin);
p_spsc = save_p_spsc;
// win_goto() may trigger an autocommand that already closes the // win_goto() may trigger an autocommand that already closes the
// cmdline window. // cmdline window.
@ -4662,6 +4659,7 @@ open_cmdwin(void)
// Restore window sizes. // Restore window sizes.
win_size_restore(&winsizes); win_size_restore(&winsizes);
skip_win_fix_cursor = FALSE;
} }
ga_clear(&winsizes); ga_clear(&winsizes);

View File

@ -1737,3 +1737,8 @@ EXTERN int channel_need_redraw INIT(= FALSE);
// While executing a regexp and set to OPTION_MAGIC_ON or OPTION_MAGIC_OFF this // While executing a regexp and set to OPTION_MAGIC_ON or OPTION_MAGIC_OFF this
// overrules p_magic. Otherwise set to OPTION_MAGIC_NOT_SET. // overrules p_magic. Otherwise set to OPTION_MAGIC_NOT_SET.
EXTERN optmagic_T magic_overruled INIT(= OPTION_MAGIC_NOT_SET); EXTERN optmagic_T magic_overruled INIT(= OPTION_MAGIC_NOT_SET);
#ifdef FEAT_CMDWIN
// Skip win_fix_cursor() call for 'nosplitscroll' when cmdwin is closed.
EXTERN int skip_win_fix_cursor INIT(= FALSE);
#endif

View File

@ -703,6 +703,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
469,
/**/ /**/
468, 468,
/**/ /**/

View File

@ -6410,6 +6410,10 @@ win_fix_cursor(int normal)
if (wp->w_buffer->b_ml.ml_line_count < wp->w_height) if (wp->w_buffer->b_ml.ml_line_count < wp->w_height)
return; return;
#ifdef FEAT_CMDWIN
if (skip_win_fix_cursor)
return;
#endif
so = MIN(wp->w_height / 2, so); so = MIN(wp->w_height / 2, so);
// Check if cursor position is above topline or below botline. // Check if cursor position is above topline or below botline.