Fix no_names and default icon

Remove display name when default icon is configured and no_names is set to true. Fixes #17
roosta · Feb 2, 2021 · 23ace5c · 23ace5c
1 parent 4868c64
commit 23ace5c
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/src/lib.rs b/src/lib.rs
@@ -129,18 +129,28 @@ fn get_class(conn: &xcb::Connection, id: u32, config: &Config) -> Result<String,
         class
     };
 
+    let no_names = get_option(&config, "no_names");
+
     // Format final result
     Ok(match config.icons.get(name) {
         Some(icon) => {
-            if get_option(&config, "no_names") {
+            if no_names {
                 format!("{}", icon)
             } else {
                 format!("{} {}", icon, display_name)
             }
         }
         None => match config.general.get("default_icon") {
-            Some(default_icon) => format!("{} {}", default_icon, display_name),
-            None => format!("{}", display_name),
+            Some(default_icon) => {
+                if no_names {
+                    format!("{}", default_icon)
+                } else {
+                    format!("{} {}", default_icon, display_name)
+                }
+            }
+            None => {
+                format!("{}", display_name)
+            }
         },
     })
 }