From 55f4f6951571d44862d5f1bc3ad7094953b788b6 Mon Sep 17 00:00:00 2001
From: lazytanuki
Date: Mon, 20 Jun 2022 17:07:32 +0200
Subject: fix: do not color health summary when stdout is piped (#2836)

* fix: do not color health summary when stdout is piped

* fix: use crossterm instead of is-terminal---
 helix-term/src/health.rs | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

(limited to 'helix-term/src')

diff --git a/helix-term/src/health.rs b/helix-term/src/health.rs
index bd74f478..f64e121d 100644
--- a/helix-term/src/health.rs
+++ b/helix-term/src/health.rs
@@ -1,4 +1,7 @@
-use crossterm::style::{Color, Print, Stylize};
+use crossterm::{
+    style::{Color, Print, Stylize},
+    tty::IsTty,
+};
 use helix_core::config::{default_syntax_loader, user_syntax_loader};
 use helix_loader::grammar::load_runtime_file;
 use std::io::Write;
@@ -106,17 +109,19 @@ pub fn languages_all() -> std::io::Result<()> {
 
     let terminal_cols = crossterm::terminal::size().map(|(c, _)| c).unwrap_or(80);
     let column_width = terminal_cols as usize / headings.len();
+    let is_terminal = std::io::stdout().is_tty();
 
     let column = |item: &str, color: Color| {
-        let data = format!(
+        let mut data = format!(
             "{:width$}",
             item.get(..column_width - 2)
                 .map(|s| format!("{}…", s))
                 .unwrap_or_else(|| item.to_string()),
             width = column_width,
-        )
-        .stylize()
-        .with(color);
+        );
+        if is_terminal {
+            data = data.stylize().with(color).to_string();
+        }
 
         // We can't directly use println!() because of
         // https://github.com/crossterm-rs/crossterm/issues/589
-- 
cgit v1.2.3-70-g09d2