From 31de54096243fb000fa6e7008a914861293482a5 Mon Sep 17 00:00:00 2001
From: srdusr <99972264+srdusr@users.noreply.github.com>
Date: Fri, 9 Aug 2024 09:36:00 +0200
Subject: 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).
---
 docs/DEFAULTS.md | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'docs')

diff --git a/docs/DEFAULTS.md b/docs/DEFAULTS.md
index 373ac2e..b5bf947 100644
--- a/docs/DEFAULTS.md
+++ b/docs/DEFAULTS.md
@@ -27,9 +27,11 @@ srd.set("general.auto_focus", true)                    -- Default: true
 srd.set("monitor.primary_layout", "dynamic")           -- Default: "dynamic"
 srd.set("monitor.secondary_layout", "tiling")          -- Default: "tiling"
 srd.set("monitor.auto_detect", true)                   -- Default: true
-srd.set("monitor.primary_workspace", 1)                -- Default: 1
-srd.set("monitor.workspace_count", 10)                 -- Default: 10
 ```
+srdwm has one flat workspace list shared by every monitor, not an
+independent set per monitor - there is no "this monitor's primary
+workspace"/"this monitor's workspace count" to configure. `workspace.count`
+below is the actual knob.
 
 ### Window Behavior (`window.*`)
 ```lua
@@ -47,10 +49,10 @@ srd.set("window.remember_state", true)                 -- Default: true
 ```lua
 srd.set("workspace.count", 10)                         -- Default: 10
 srd.set("workspace.names", {"1", "2", "3", "4", "5", "6", "7", "8", "9", "0"})
-srd.set("workspace.auto_switch", false)                -- Default: false
-srd.set("workspace.persistent", true)                  -- Default: true
 srd.set("workspace.auto_back_and_forth", false)        -- Default: false
 ```
+(`workspace.auto_switch`/`workspace.persistent` are not implemented - not
+listed here since setting either currently does nothing.)
 
 ### Performance Settings (`performance.*`)
 ```lua
-- 
cgit v1.2.3