feat: hide logo if terminal size is too wide

a new CLI option to configure whether the logo should be shown
if configued to auto, Onefetch will detect the terminal size and
hide/show the logo based off that.

Signed-off-by: Luke-zhang-04 <[email protected]>

Author: Luke-zhang-04

Cargo.lock

@@ -28,6 +28,7 @@ askalono = "0.4.3"
 bytecount = "0.6.1"
 clap = "2.33.3"
 strum = { version = "0.20.0", features = ["derive"] }
+term_size = "0.3.2"
 image = "0.23.12"
 regex = "1"
 error-chain = "0.12"

Cargo.toml

@@ -7,9 +7,11 @@ use {
         language::Language,
     },
     clap::{crate_description, crate_name, crate_version, App, AppSettings, Arg},
+    colored::Colorize,
     image::DynamicImage,
     std::{convert::From, env, str::FromStr},
     strum::IntoEnumIterator,
+    term_size,
 };
 
 pub struct Cli {
@@ -220,17 +222,38 @@ impl Cli {
         .arg(
             Arg::with_name("off")
                 .long("off")
-                .help("Only shows the info lines.")
-                .conflicts_with_all(&["image", "ascii-language", "ascii-input"]),
-            ).get_matches();
+                .help("Only shows the info lines. DEPRECATED: use \"--hide-logo\" true instead.")
+                .conflicts_with_all(&["image", "ascii-language", "ascii-input", "hide-logo"]),
+            )
+        .arg(
+            Arg::with_name("hide-logo")
+                .long("hide-logo")
+                .value_name("auto | true | false")
+                .takes_value(true)
+                .max_values(1)
+                .multiple(false)
+                .default_value("auto")
+                .possible_values(&["auto", "true", "false"])
+                .help("Will hide the logo if true. If set to auto, the logo will be hidden if the terminal size is too small. If set to false, the logo will be shown no matter what.")
+                .conflicts_with("off")
+        ).get_matches();
 
         let no_bold = matches.is_present("no-bold");
         let no_merges = matches.is_present("no-merge-commits");
         let no_color_palette = matches.is_present("no-color-palette");
         let print_languages = matches.is_present("languages");
         let print_package_managers = matches.is_present("package-managers");
-        let art_off = matches.is_present("off");
+        let mut art_off = matches.is_present("off");
         let true_color = cli_utils::is_truecolor_terminal();
+        let max_term_size = 95;
+
+        // The --off flag was passed in
+        if art_off {
+            std::println!(
+                "{}",
+                ("The --off option is deprecated. Use \"--hide-logo true\" instead.").yellow(),
+            );
+        }
 
         let fields_to_hide: Vec<String> = if let Some(values) = matches.values_of("disable-fields")
         {
@@ -255,6 +278,25 @@ impl Cli {
             None
         };
 
+        if let Some(should_hide_logo) = matches.value_of("hide-logo") {
+            if should_hide_logo == "true" {
+                art_off = true;
+            } else if should_hide_logo == "false" {
+                art_off = false;
+            } else {
+                if let Some((width, _)) = term_size::dimensions_stdout() {
+                    art_off = width <= max_term_size;
+                } else {
+                    std::println!(
+                        "{}",
+                        ("Coult not get terminal width. ASCII art will be displayed.").yellow(),
+                    );
+
+                    art_off = false;
+                }
+            }
+        }
+
         if image.is_some() && image_backend.is_none() {
             return Err("Could not detect a supported image backend".into());
         }