Mercurial > vim
diff src/terminal.c @ 12973:418941f0df08 v8.0.1362
patch 8.0.1362: terminal window colors wrong when using Terminal highlighting
commit https://github.com/vim/vim/commit/a7c54cfcf825e8e99db03f4ccdb1a32cd0714c52
Author: Bram Moolenaar <Bram@vim.org>
Date: Fri Dec 1 21:07:20 2017 +0100
patch 8.0.1362: terminal window colors wrong when using Terminal highlighting
Problem: Terminal window colors wrong when using Terminal highlighting.
Solution: Set ansi_index when setting the default color. Also cache the
color index for Terminal. (Ozaki Kiichi, closes #2393)
author | Christian Brabandt <cb@256bit.org> |
---|---|
date | Fri, 01 Dec 2017 21:15:05 +0100 |
parents | a9f6a874b64f |
children | fc0d4a036654 |
line wrap: on
line diff
--- a/src/terminal.c +++ b/src/terminal.c @@ -188,6 +188,9 @@ static void term_free_vterm(term_T *term * backspace key. */ static int term_backspace_char = BS; +/* "Terminal" highlight group colors. */ +static int term_default_cterm_fg = -1; +static int term_default_cterm_bg = -1; /************************************** * 1. Generic code for all systems. @@ -1834,20 +1837,12 @@ cell2attr(VTermScreenCellAttrs cellattrs int bg = color2index(&cellbg, FALSE, &bold); /* Use the "Terminal" highlighting for the default colors. */ - if (fg == 0 || bg == 0) + if ((fg == 0 || bg == 0) && t_colors >= 16) { - int id = syn_name2id((char_u *)"Terminal"); - - if (id != 0 && t_colors >= 16) - { - int cterm_fg, cterm_bg; - - syn_id2cterm_bg(id, &cterm_fg, &cterm_bg); - if (cterm_fg >= 0) - fg = cterm_fg + 1; - if (cterm_bg >= 0) - bg = cterm_bg + 1; - } + if (fg == 0 && term_default_cterm_fg >= 0) + fg = term_default_cterm_fg + 1; + if (bg == 0 && term_default_cterm_bg >= 0) + bg = term_default_cterm_bg + 1; } /* with 8 colors set the bold attribute to get a bright foreground */ @@ -2470,6 +2465,7 @@ cterm_color2rgb(int nr, VTermColor *rgb) rgb->blue = cube_value[idx % 6]; rgb->green = cube_value[idx / 6 % 6]; rgb->red = cube_value[idx / 36 % 6]; + rgb->ansi_index = VTERM_ANSI_INDEX_NONE; } else if (nr < 256) { @@ -2478,6 +2474,7 @@ cterm_color2rgb(int nr, VTermColor *rgb) rgb->blue = grey_ramp[idx]; rgb->green = grey_ramp[idx]; rgb->red = grey_ramp[idx]; + rgb->ansi_index = VTERM_ANSI_INDEX_NONE; } } @@ -2520,6 +2517,7 @@ create_vterm(term_T *term, int rows, int } fg->red = fg->green = fg->blue = fgval; bg->red = bg->green = bg->blue = bgval; + fg->ansi_index = bg->ansi_index = VTERM_ANSI_INDEX_DEFAULT; /* The "Terminal" highlight group overrules the defaults. */ id = syn_name2id((char_u *)"Terminal"); @@ -2582,13 +2580,10 @@ create_vterm(term_T *term, int rows, int #endif if (id != 0 && t_colors >= 16) { - int cterm_fg, cterm_bg; - - syn_id2cterm_bg(id, &cterm_fg, &cterm_bg); - if (cterm_fg >= 0) - cterm_color2rgb(cterm_fg, fg); - if (cterm_bg >= 0) - cterm_color2rgb(cterm_bg, bg); + if (term_default_cterm_fg >= 0) + cterm_color2rgb(term_default_cterm_fg, fg); + if (term_default_cterm_bg >= 0) + cterm_color2rgb(term_default_cterm_bg, bg); } else { @@ -2705,6 +2700,16 @@ set_ref_in_term(int copyID) } /* + * Cache "Terminal" highlight group colors. + */ + void +set_terminal_default_colors(int cterm_fg, int cterm_bg) +{ + term_default_cterm_fg = cterm_fg - 1; + term_default_cterm_bg = cterm_bg - 1; +} + +/* * Get the buffer from the first argument in "argvars". * Returns NULL when the buffer is not for a terminal window. */