Skip to content

Commit

Permalink
patch 9.0.1498: in a terminal window the cursor may jump around
Browse files Browse the repository at this point in the history
Problem:    In a terminal window the cursor may jump around. (Kenny Stauffer)
Solution:   Do not move the cursor to the position for terminal-normal mode.
            (closes #12312)
  • Loading branch information
brammool committed Apr 29, 2023
1 parent 971cd2b commit 58806c1
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 18 deletions.
45 changes: 27 additions & 18 deletions src/terminal.c
Expand Up @@ -1208,6 +1208,24 @@ term_write_job_output(term_T *term, char_u *msg_arg, size_t len_arg)
vterm_screen_flush_damage(vterm_obtain_screen(vterm));
}

static void
position_cursor(win_T *wp, VTermPos *pos)
{
wp->w_wrow = MIN(pos->row, MAX(0, wp->w_height - 1));
wp->w_wcol = MIN(pos->col, MAX(0, wp->w_width - 1));
#ifdef FEAT_PROP_POPUP
if (popup_is_popup(wp))
{
wp->w_wrow += popup_top_extra(wp);
wp->w_wcol += popup_left_extra(wp);
wp->w_flags |= WFLAG_WCOL_OFF_ADDED | WFLAG_WROW_OFF_ADDED;
}
else
wp->w_flags &= ~(WFLAG_WCOL_OFF_ADDED | WFLAG_WROW_OFF_ADDED);
#endif
wp->w_valid |= (VALID_WCOL|VALID_WROW);
}

static void
update_cursor(term_T *term, int redraw)
{
Expand All @@ -1219,7 +1237,16 @@ update_cursor(term_T *term, int redraw)
term->tl_cursor_pos.col);
else
#endif
if (!term_job_running(term))
// avoid the cursor positioned below the last used line
setcursor();
else
{
// do not use the window cursor position
position_cursor(curwin, &curbuf->b_term->tl_cursor_pos);
windgoto(W_WINROW(curwin) + curwin->w_wrow,
curwin->w_wincol + curwin->w_wcol);
}
if (redraw)
{
aco_save_T aco;
Expand Down Expand Up @@ -2358,24 +2385,6 @@ send_keys_to_term(term_T *term, int c, int modmask, int typed)
return OK;
}

static void
position_cursor(win_T *wp, VTermPos *pos)
{
wp->w_wrow = MIN(pos->row, MAX(0, wp->w_height - 1));
wp->w_wcol = MIN(pos->col, MAX(0, wp->w_width - 1));
#ifdef FEAT_PROP_POPUP
if (popup_is_popup(wp))
{
wp->w_wrow += popup_top_extra(wp);
wp->w_wcol += popup_left_extra(wp);
wp->w_flags |= WFLAG_WCOL_OFF_ADDED | WFLAG_WROW_OFF_ADDED;
}
else
wp->w_flags &= ~(WFLAG_WCOL_OFF_ADDED | WFLAG_WROW_OFF_ADDED);
#endif
wp->w_valid |= (VALID_WCOL|VALID_WROW);
}

/*
* Handle CTRL-W "": send register contents to the job.
*/
Expand Down
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -695,6 +695,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
1498,
/**/
1497,
/**/
Expand Down

0 comments on commit 58806c1

Please sign in to comment.