Allow workspace names to be changed dynamically (#904)

* Add un/set workspace name actions

* Add SetWorkspaceName reference to proptests

* Simplify unname_workspace

* Add ewaf version of set first workspace name test

* Simplify more

* Fix comment

* Make workspace in set-workspace-name a positional option

---------

Co-authored-by: Ivan Molodetskikh <yalterz@gmail.com>

1 files changed, 7 insertions, 12 deletions

diff --git a/src/layout/monitor.rs b/src/layout/monitor.rs
index 57ca7f53..70009d67 100644
--- a/src/layout/monitor.rs
+++ b/src/layout/monitor.rs
@@ -382,18 +382,13 @@ impl<W: LayoutElement> Monitor<W> {
         }
     }
 
-    pub fn unname_workspace(&mut self, workspace_name: &str) -> bool {
-        for ws in &mut self.workspaces {
-            if ws
-                .name
-                .as_ref()
-                .map_or(false, |name| name.eq_ignore_ascii_case(workspace_name))
-            {
-                ws.unname();
-                return true;
-            }
-        }
-        false
+    pub fn unname_workspace(&mut self, id: WorkspaceId) -> bool {
+        let Some(ws) = self.workspaces.iter_mut().find(|ws| ws.id() == id) else {
+            return false;
+        };
+
+        ws.unname();
+        true
     }
 
     pub fn move_left(&mut self) -> bool {