@@ -26,7 +26,8 @@ impl OutputType {
2626
2727 /// Try to launch the pager. Fall back to stdout in case of errors.
2828 fn try_pager(quit_if_one_screen: bool, pager_from_config: Option<&str>) -> Result<Self> {
29− let pager_from_env = env::var("BAT_PAGER").or_else(|_| env::var("PAGER"));
29+ let pager_from_env =
30+ env::var("BAT_PAGER").or_else(|_| env::var("PAGER").map(add_default_flags_to_less));
3031
3132 let pager = pager_from_config
3233 .map(|p| p.to_string())
@@ -87,6 +88,28 @@ impl OutputType {
8788 }
8889}
8990
91+// Since someone could set PAGER to less without arg -R that bat needs,
92+// ignore flags for less and change flags to -FRX.
93+// If a user wants an env variable with flags unchanged, he/she should use
94+// BAT_PAGER.
95+fn add_default_flags_to_less(pager: String) -> String {
96+ if let Some(pager_name) = shell_words::split(&pager)
97+ .ok()
98+ .as_ref()
99+ .and_then(|flags| flags.first())
100+ .and_then(|pager_path| {
101+ let path_buf = PathBuf::from(pager_path);
102+ path_buf.file_stem().map(|os_str| os_str.to_os_string())
103+ })
104+ {
105+ if pager_name == OsString::from("less") {
106+ return "less -FRX".to_string();
107+ }
108+ }
109+
110+ pager.to_string()
111+}
112+
90113impl Drop for OutputType {
91114 fn drop(&mut self) {
92115 if let OutputType::Pager(ref mut command) = *self {