wez/wezterm
1
1
Fork 0
mirror of https://github.com/wez/wezterm.git synced 2024-08-16 09:40:34 +03:00
Code Issues Projects Releases Wiki Activity

termwiz: revert to using semicolon when encoding 8 bit SGR colors

Browse Source 
Windows does not seem to support 8 bit colors using the colon
separator. The intention of e1f7edaeb3
seems to be around true color, so I left that alone and only changed 8
bit color back to semicolons.

Fixes #2723
This commit is contained in:
Muir Manders 2022-11-09 11:42:37 -08:00
parent c71e22e57d
commit 5aff4e5f07
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
termwiz/src/escape/csi.rs
View File

@ -1402,7 +1402,7 @@ impl Display for Sgr {
$(AnsiColor::$Ansi => code!($code) ,)*
}
} else {
write!(f, "{}:5:{}m", SgrCode::$eightbit as i64, $idx)?
write!(f, "{};5;{}m", SgrCode::$eightbit as i64, $idx)?
}
}
}
@ -1564,7 +1564,7 @@ impl Display for Sgr {
}
}
Sgr::UnderlineColor(ColorSpec::PaletteIndex(idx)) => {
write!(f, "{}:5:{}m", SgrCode::UnderlineColor as i64, *idx)?
write!(f, "{};5;{}m", SgrCode::UnderlineColor as i64, *idx)?
}
}
Ok(())
@ -2879,7 +2879,7 @@ mod test {
)))]
);
assert_eq!(
parse('m', &[58, 5, 220, 255, 255], "\x1b[58:5:220m\x1b[255;255m"),
parse('m', &[58, 5, 220, 255, 255], "\x1b[58;5;220m\x1b[255;255m"),
vec![
CSI::Sgr(Sgr::UnderlineColor(ColorSpec::PaletteIndex(220))),
CSI::Unspecified(Box::new(Unspecified {
@ -2919,7 +2919,7 @@ mod test {
)))]
);
assert_eq!(
parse('m', &[38, 5, 220, 255, 255], "\x1b[38:5:220m\x1b[255;255m"),
parse('m', &[38, 5, 220, 255, 255], "\x1b[38;5;220m\x1b[255;255m"),
vec![
CSI::Sgr(Sgr::Foreground(ColorSpec::PaletteIndex(220))),
CSI::Unspecified(Box::new(Unspecified {