2783 setting terminal title

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Features
 
+- Set terminal title to file names when Paging is not Paging::Never #2807 (@Oliver-Looney)
+
 ## Bugfixes
 
 - Fix `NO_COLOR` support, see #2767 (@acuteenvy)

src/bin/bat/main.rs

@@ -227,9 +227,24 @@ pub fn list_themes(cfg: &Config, config_dir: &Path, cache_dir: &Path) -> Result<
     Ok(())
 }
 
+fn set_terminal_title_to_inputs_names(inputs: &Vec<Input>) {
+    let mut input_names = "bat: ".to_string();
+    for (index, input) in inputs.iter().enumerate() {
+        input_names += &input.description.name.to_string();
+        if index < inputs.len() - 1 {
+            input_names += ", ";
+        }
+    }
+    print!("\x1b]2;{}\x07", input_names);
+    io::stdout().flush().unwrap();
+}
+
 fn run_controller(inputs: Vec<Input>, config: &Config, cache_dir: &Path) -> Result<bool> {
     let assets = assets_from_cache_or_binary(config.use_custom_assets, cache_dir)?;
     let controller = Controller::new(config, &assets);
+    if config.paging_mode != PagingMode::Never {
+        set_terminal_title_to_inputs_names(&inputs);
+    }
     controller.run(inputs, None)
 }
 

src/input.rs

@@ -13,7 +13,7 @@ use crate::error::*;
 /// This tells bat how to refer to the input.
 #[derive(Clone)]
 pub struct InputDescription {
-    pub(crate) name: String,
+    pub name: String,
 
     /// The input title.
     /// This replaces the name if provided.
@@ -94,7 +94,7 @@ pub(crate) struct InputMetadata {
 pub struct Input<'a> {
     pub(crate) kind: InputKind<'a>,
     pub(crate) metadata: InputMetadata,
-    pub(crate) description: InputDescription,
+    pub description: InputDescription,
 }
 
 pub(crate) enum OpenedInputKind {

tests/integration_tests.rs

@@ -653,7 +653,7 @@ fn pager_disable() {
         .arg("test.txt")
         .assert()
         .success()
-        .stdout(predicate::eq("hello world\n").normalize());
+        .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07hello world\n").normalize());
 }
 
 #[test]
@@ -734,7 +734,7 @@ fn env_var_pager_value_bat() {
         .arg("test.txt")
         .assert()
         .success()
-        .stdout(predicate::eq("hello world\n").normalize());
+        .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07hello world\n").normalize());
 }
 
 #[test]
@@ -772,7 +772,7 @@ fn pager_most_from_pager_env_var() {
             .arg("test.txt")
             .assert()
             .success()
-            .stdout(predicate::eq("hello world\n").normalize());
+            .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07hello world\n").normalize());
     });
 }
 
@@ -818,7 +818,7 @@ fn pager_most_with_arg() {
             .arg("test.txt")
             .assert()
             .success()
-            .stdout(predicate::eq("hello world\n").normalize());
+            .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07hello world\n").normalize());
     });
 }
 
@@ -833,7 +833,7 @@ fn pager_more() {
             .arg("test.txt")
             .assert()
             .success()
-            .stdout(predicate::eq("hello world\n").normalize());
+            .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07hello world\n").normalize());
     });
 }
 
@@ -896,7 +896,7 @@ fn enable_pager_if_disable_paging_flag_comes_before_paging() {
         .arg("test.txt")
         .assert()
         .success()
-        .stdout(predicate::eq("pager-output\n").normalize());
+        .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07pager-output\n").normalize());
 }
 
 #[test]
@@ -908,7 +908,7 @@ fn enable_pager_if_pp_flag_comes_before_paging() {
         .arg("test.txt")
         .assert()
         .success()
-        .stdout(predicate::eq("pager-output\n").normalize());
+        .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07pager-output\n").normalize());
 }
 
 #[test]

tests/system_wide_config.rs

@@ -8,11 +8,9 @@ use utils::command::bat_with_config;
 #[test]
 #[ignore]
 fn use_systemwide_config() {
-    bat_with_config()
-        .arg("test.txt")
-        .assert()
-        .success()
-        .stdout(predicate::eq("dummy-pager-from-system-config\n").normalize());
+    bat_with_config().arg("test.txt").assert().success().stdout(
+        predicate::eq("\u{1b}]2;bat: test.txt\x07dummy-pager-from-system-config\n").normalize(),
+    );
 }
 
 // This test is ignored, as it needs a special system wide config put into place
@@ -25,5 +23,5 @@ fn config_overrides_system_config() {
         .arg("test.txt")
         .assert()
         .success()
-        .stdout(predicate::eq("dummy-pager-from-config\n").normalize());
+        .stdout(predicate::eq("\u{1b}]2;bat: test.txt\x07dummy-pager-from-config\n").normalize());
 }