Fix a misleading workspace comment and drop four dead config keys

visible_windows' doc comment claimed windows show "on the active
workspace of whichever monitor they're assigned to" - the code never
reads w.monitor at all; current_workspace is one flat value shared by
every monitor, not per-output. Documented that explicitly on both the
field and the method, since this is a real behavioral difference from
Hyprland worth a reader actually seeing, not just an inaccurate comment
to fix quietly.

monitor.primary_workspace/monitor.workspace_count describe a per-
monitor-workspace design that doesn't exist; workspace.auto_switch/
workspace.persistent were never wired to any behavior. All four were
seeded into default_config() and documented in DEFAULTS.md, so
srd.set()/srd.get() on them silently succeeded while doing nothing --
removed from both, matching the precedent already set by general.
rounded_corners' deliberate absence from default_config for a
different reason (backend-dependent default rather than unbuilt).

1 files changed, 6 insertions, 1 deletions

diff --git a/crates/core/src/manager/workspaces.rs b/crates/core/src/manager/workspaces.rs
index 485ced0..6ffb21b 100644
--- a/crates/core/src/manager/workspaces.rs
+++ b/crates/core/src/manager/workspaces.rs
@@ -75,7 +75,12 @@ impl WindowManager {
     }
 
     /// Windows that should currently be shown to the user: those on the
-    /// active workspace of whichever monitor they're assigned to, and not minimized.
+    /// current workspace, and not minimized.
+    ///
+    /// `current_workspace` is a single value shared by every monitor --
+    /// srdwm does not have Hyprland-style independent per-monitor
+    /// workspaces, so switching workspace changes what's shown on every
+    /// screen at once. `w.monitor` plays no part in this filter at all.
     pub fn visible_windows(&self) -> impl Iterator<Item = &Window> {
         self.windows.values().filter(|w| w.workspace == self.current_workspace && !w.minimized)
     }