Change allOutputs default to "yes"

Changes the default behavior to show windows from all outputs instead
of only the cursor's output.

- Updated default value in documentation from "no" to "yes"
- Inverted parsing logic: allOutputs="no" now enables output filtering
- Removed LAB_VIEW_CRITERIA_CURSOR_OUTPUT from default criteria flags

Users who prefer the previous behavior (output filtering) can now set
allOutputs="no" in their configuration.
This commit is contained in:
Samet Aylak 2025-11-02 12:14:19 -05:00
parent d1e96727cb
commit 50a0602b7a
No known key found for this signature in database
GPG key ID: E5DEF544B937EE49
3 changed files with 8 additions and 10 deletions

View file

@ -339,7 +339,7 @@ this is for compatibility with Openbox.
## WINDOW SWITCHER ## WINDOW SWITCHER
``` ```
<windowSwitcher show="yes" style="classic" preview="yes" outlines="yes" allWorkspaces="no" allOutputs="no"> <windowSwitcher show="yes" style="classic" preview="yes" outlines="yes" allWorkspaces="no" allOutputs="yes">
<fields> <fields>
<field content="icon" width="5%" /> <field content="icon" width="5%" />
<field content="desktop_entry_name" width="30%" /> <field content="desktop_entry_name" width="30%" />
@ -367,8 +367,7 @@ this is for compatibility with Openbox.
are shown). are shown).
*allOutputs* [yes|no] Show windows regardless of what output *allOutputs* [yes|no] Show windows regardless of what output
they are on. Default no (that is only windows on the cursor output they are on. Default yes.
are shown).
*unshade* [yes|no] Temporarily unshade windows when switching between *unshade* [yes|no] Temporarily unshade windows when switching between
them and permanently unshade on the final selection. Default is yes. them and permanently unshade on the final selection. Default is yes.

View file

@ -78,7 +78,7 @@
</theme> </theme>
<windowSwitcher show="yes" style="classic" preview="yes" <windowSwitcher show="yes" style="classic" preview="yes"
outlines="yes" allWorkspaces="no" allOutputs="no" unshade="yes"> outlines="yes" allWorkspaces="no" allOutputs="yes" unshade="yes">
<fields> <fields>
<field content="icon" width="5%" /> <field content="icon" width="5%" />
<field content="desktop_entry_name" width="30%" /> <field content="desktop_entry_name" width="30%" />
@ -98,7 +98,7 @@
Some contents are fixed-length and others are variable-length. Some contents are fixed-length and others are variable-length.
See "man 5 labwc-config" for details. See "man 5 labwc-config" for details.
<windowSwitcher show="yes" preview="no" outlines="no" allWorkspaces="yes" allOutputs="no"> <windowSwitcher show="yes" preview="no" outlines="no" allWorkspaces="yes" allOutputs="yes">
<fields> <fields>
<field content="workspace" width="5%" /> <field content="workspace" width="5%" />
<field content="state" width="3%" /> <field content="state" width="3%" />
@ -118,7 +118,7 @@
then workspace name, then identifier/app-id, then the window title. then workspace name, then identifier/app-id, then the window title.
It uses 100% of OSD window width. It uses 100% of OSD window width.
<windowSwitcher show="yes" preview="no" outlines="no" allWorkspaces="yes" allOutputs="no"> <windowSwitcher show="yes" preview="no" outlines="no" allWorkspaces="yes" allOutputs="yes">
<fields> <fields>
<field content="custom" format="foobar %b %3s %-10o %-20W %-10i %t" width="100%" /> <field content="custom" format="foobar %b %3s %-10o %-20W %-10i %t" width="100%" />
</fields> </fields>

View file

@ -1218,9 +1218,9 @@ entry(xmlNode *node, char *nodename, char *content)
~LAB_VIEW_CRITERIA_CURRENT_WORKSPACE; ~LAB_VIEW_CRITERIA_CURRENT_WORKSPACE;
} }
} else if (!strcasecmp(nodename, "allOutputs.windowSwitcher")) { } else if (!strcasecmp(nodename, "allOutputs.windowSwitcher")) {
if (parse_bool(content, -1) == true) { if (parse_bool(content, -1) == false) {
rc.window_switcher.criteria &= rc.window_switcher.criteria |=
~LAB_VIEW_CRITERIA_CURSOR_OUTPUT; LAB_VIEW_CRITERIA_CURSOR_OUTPUT;
} }
} else if (!strcasecmp(nodename, "unshade.windowSwitcher")) { } else if (!strcasecmp(nodename, "unshade.windowSwitcher")) {
set_bool(content, &rc.window_switcher.unshade); set_bool(content, &rc.window_switcher.unshade);
@ -1438,7 +1438,6 @@ rcxml_init(void)
rc.window_switcher.outlines = true; rc.window_switcher.outlines = true;
rc.window_switcher.unshade = true; rc.window_switcher.unshade = true;
rc.window_switcher.criteria = LAB_VIEW_CRITERIA_CURRENT_WORKSPACE rc.window_switcher.criteria = LAB_VIEW_CRITERIA_CURRENT_WORKSPACE
| LAB_VIEW_CRITERIA_CURSOR_OUTPUT
| LAB_VIEW_CRITERIA_ROOT_TOPLEVEL | LAB_VIEW_CRITERIA_ROOT_TOPLEVEL
| LAB_VIEW_CRITERIA_NO_SKIP_WINDOW_SWITCHER; | LAB_VIEW_CRITERIA_NO_SKIP_WINDOW_SWITCHER;