fix image backend detection

Author: o2sh

Diff for: src/onefetch/image_backends/kitty.rs

@@ -5,7 +5,6 @@ use {
         c_void, ioctl, poll, pollfd, read, tcgetattr, tcsetattr, termios, winsize, ECHO, ICANON,
         POLLIN, STDIN_FILENO, STDOUT_FILENO, TCSANOW, TIOCGWINSZ,
     },
-    std::env,
     std::io::{stdout, Write},
     std::time::Instant,
 };
@@ -18,10 +17,6 @@ impl KittyBackend {
     }
 
     pub fn supported() -> bool {
-        if !env::var("KITTY_WINDOW_ID").unwrap_or_else(|_| "".to_string()).is_empty() {
-            return true;
-        }
-
         // save terminal attributes and disable canonical input processing mode
         let old_attributes = unsafe {
             let mut old_attributes: termios = std::mem::zeroed();

Diff for: src/onefetch/image_backends/mod.rs

@@ -12,17 +12,20 @@ pub trait ImageBackend {
     fn add_image(&self, lines: Vec<String>, image: &DynamicImage, colors: usize) -> Result<String>;
 }
 
-#[cfg(not(windows))]
 pub fn get_best_backend() -> Option<Box<dyn ImageBackend>> {
-    if sixel::SixelBackend::supported() {
-        Some(Box::new(sixel::SixelBackend::new()))
-    } else if kitty::KittyBackend::supported() {
+    #[cfg(not(windows))]
+    if kitty::KittyBackend::supported() {
         Some(Box::new(kitty::KittyBackend::new()))
     } else if iterm::ITermBackend::supported() {
         Some(Box::new(iterm::ITermBackend::new()))
+    } else if sixel::SixelBackend::supported() {
+        Some(Box::new(sixel::SixelBackend::new()))
     } else {
         None
     }
+
+    #[cfg(windows)]
+    None
 }
 
 pub fn get_image_backend(backend_name: &str) -> Option<Box<dyn ImageBackend>> {
@@ -33,12 +36,8 @@ pub fn get_image_backend(backend_name: &str) -> Option<Box<dyn ImageBackend>> {
         "sixel" => Box::new(sixel::SixelBackend::new()) as Box<dyn ImageBackend>,
         _ => unreachable!(),
     });
+
     #[cfg(windows)]
     let backend = None;
     backend
 }
-
-#[cfg(windows)]
-pub fn get_best_backend() -> Option<Box<dyn ImageBackend>> {
-    None
-}