Move border-struct to common/border.h for sharing

view: add minimize method

...and call wlr_xwayland_surface_set_minimized() for xwayland surfaces on
(un)minimize.

Fixes: #958

config: support <desktops number=""> update documentation

config: support <desktops number="">

config: support <desktops number=>

config: support <desktops number=> update documentation

config: support <desktops number=>

Documentation updated at  and

config: support <desktops number="">

docs/rc.xml.all: fix mousebind button casing

config: fix bug with default scroll mousebind

Specify attribute name when adding default mousebind actions.

This is required because in src/action.c action GoToDesktop requires an
argument with key 'to' and does not accept `action_get_first_arg()`
because the action supports other optional attributes so we could not
guarantee that 'to' is the first one.

Fixes: #948

dnd: refocus surface under cursor on drag-destroy

Keyboard focus is not changed during drag, so we need to refocus the
surface under the cursor and the end of a drag-and-drop operation.

Fixes issue #939

docs: fix some typos

Between, output, rounded and focused.

Added one inconsistently missing trailing period.

Inserted newline before last word in one line in
labwc-actions.5.scd in order to keep that line from
passing 80-column boundary.

window-rules: add skipTaskbar and skipWindowSwitcher

action: add ToggleAlwaysOnBottom

xdg-popup: render above always-on-top

action: support {Go,Send}ToDesktop 'wrap' option

Make wrap 'true' by default for both GoToDesktop and SendToDesktop,
in order to default to the current behaviour, and to stay consistent
with Openbox behaviour.

src/output.c: destroy workspace OSD on output destroy

src/workspaces.c: do not render OSD on disabled outputs

Fixes #914

src/action.c: validate action type when assigning arguments

Before this patch it was possible to assign an argument with an
existing name to an action that didn't support the given argument.
An example of this is using `direction` for `GoToDesktop`.

This patch now only creates action arguments that are actually
defined for the given action type and logs an error for
unsupported arguments.

The commit also makes sure to always supply the argument name.
This will reduce required checks in other parts of the codebase
in future commits.

Partly fixes: #894

includes: identifier consistency in include guards

Made all header files to have LABWC_ prefix in include guard identifers.

Converted from __LABWC_ in 35 include/ files.
Converted from __LAB_ in 5 include/ files.
Added LABWC prefix to 3 include/ files.
Added include guards to 3 include/ files.

The double underscores were removed since according to C standard
those "are always reserved for any use".

src/config/rcxml.c: move region validation into the validate() hook

labwc-config(5): describe element/attribute-agnostic XML parsing

window-rules: support title criteria

Example config:

    <windowRules>
      <windowRule identifier="foot" title="max">
        <action name="Maximize"/>
      </windowRule>
    </windowRules>

Observe that:

- `foot -T foo` starts maximized
- `xterm -T foo` starts normal

build: bump version to 0.6.3

NEWS.md: add notes on 0.6.3

xwayland: Add mapped argument to xwayland_view_create()

This is a trivial cleanup to make xwayland_view_create() symmetrical with
xwayland_unmanaged_create(), and avoid the need to access view->impl from
xwayland-unmanaged.c.

The return value of xwayland_view_create() is no longer user, so return void.

No functional change.

followMouse: add followMouseRequiresMovement

This implements the same config option as `underMouse` in Openbox.

Fixes #862

match.c: fix g_utf8_casefold() memory leak

Reported-by: @jlindgren90

Support window-rules

Two types of window rules are supported, actions and properties. They are
defined as shown below.

    <windowRules>

      <!-- Action -->
      <windowRule identifier="some-application">
        <action name="Maximize"/>
      </windowRule>

      <!-- Property -->
      <windowRule identifier="foo*" serverDecoration="yes|no"/>

    </windowRules>

Rules are applied if windows match the criteria defined by the
'identifier' attribute which relates to app_id for native Wayland windows
and WM_CLASS for XWayland clients.

Matching against patterns with '*' (wildcard) and '?' (joker) is
supported.

Add 'serverDecoration' property.

action: add Maximize

Add common/match.c with match_glob() from theme.c

...and make it fully case-insensitive in preparation for window-rules.

src/output.c: fix adaptive sync

Fixes #642

Reported-by: @heroin-moose
Tested-by: @heroin-moose

parse-bool.c: add helpers to set bool/int iff valid boolean

Ignore SIGPIPE

Prevent wayland clients that request the X11 clipboard but closing
their read fd prematurely to crash labwc because of the unhandled
SIGPIPE signal. It is caused by wlroots trying to write the X11
clipboard data to the closed fd of the wayland client.

See https://github.com/labwc/labwc/issues/890#issuecomment-1524962995
for a reproducer involving xclip and wl-paste | head -c 1.

Fixes #890
Reported-by: @FuzzyQuills

output: Make the session lock layer the topmost

Make sure that only the session lock layer is visible to a user. Fixes
the issue with mako notifications being rendered above the swaylock
window.

action: gracefully handle argument without key

Fixes: issue #894

common/parse-bool.c: make parse_bool() generic

...to avoid multiple versions of a boolean-parser.

- Optionally take a default value
- Return -1 on error
- Rename get-bool.c to parse-bool.c

action: add get_arg_value_{str,bool} to simplify multi-arg handling

action: make action_arg_add_bool() static

Add Polish translation

osd: set field width to ellipsize

config: move cycle_* bools to window_switcher struct

config: support window switcher field definitions

<windowSwitcher>
  <fields>
    <field content="type" width="25%" />
    <field content="app_id" width="25%" />
    <field content="title" width="50%" />
  </fields>
</windowSwitcher>

Relates to issues #852 #855 #879

config: make <windowSwitcher> a toplevel element

...rather than a child of <core>

xwayland: validate PID before activating unmanaged surface

Check that an unmanaged surface trying to grab focus is actually a child
of the topmost mapped view.

xwayland: handle unmanaged activate requests

xwayland: handle unmanaged override-redirect requests

Fixes: issue #874

xwayland: split out xwayland_view constructor

...and make it public in preparation for supporting override-redirect
requests from unmanaged xwayland surfaces.

po: clean up language files
Changed entries in "Language" field to language code
Added language names to "Language Team"
Removed example email address
Fixed author names in Copyright section

Update es.po

Small grammatical error

Add Indonesia translation

Add Russian translation

src/cursor.c: handle pointer constraint position hint

Now cursor hints are respected once cursor movement occurs
after unlocking the pointer.

Fixes Overwatch 2 failing to center the mouse pointer when
opening the communication wheel in-game.

Fixes #872

Support ext-session-lock protocol

README.md: simplify configuration section

view: Add view_move_to_front/back().

This avoids calling view->impl functions from cursor.c and desktop.c.

v2: Add an explicit recursion guard in cursor_update_focus().

server: add wlr_backend_autocreate() trouble-shooting message

src/cursor.c: make sure the cursor is visible after startup

Before this patch, the cursor was invisible after labwc startup
unless manually moved by the user (or being positioned on top
of some some automatically spawned client surface). This patch
sets the cursor from its uninitialized value (LAB_CURSOR_CLIENT)
to the default cursor.

src/cursor.c: fix invisible cursor on output loss / restore

Previously, the cursor image was only updated on output loss when the
cursor was on a labwc owned surface. This patch forces a re-enter of a
client surface in the remaining case of cursor being over a non-labwc
surface which causes the client to re-set its own cursor image.

This fixes a regression caused by 4dc99e2f3856650f3b61778f015a823cfae978d2.
Thanks to @Flrian for finding the root cause of the issue.

Fixes #820

Reported-by: @Flrian
Tested-by: @Flrian

config: support scroll-factor

Fixes: issue #846

Implement action "Lower"

desktop: move scene-tree node in move-to-back

view_minimize() does not need to call desktop_move_to_back() because the
stacking order is not changed and the windowSwitcher uses the scene-tree
nodes anyway.

Note: Movement of xwayland sub-views still relies on keeping server->views
in sync with z-order

followMouse: allow focus switching via A-Tab

Fixes a regression caused by the recent enhancement of followMouse behavior.

Fixes: #849
Reported-by: mitigate via IRC

action: support SendToDesktop 'follow' option

Make follow 'true' by default as per Openbox 3.6 specification.
Note: this is an interface change.

Fixes: issue #841

Move get_bool() to src/common/

...in preparation for sharing it more widely

Decorations: always default to client side decorations

This is required as both decoration protocol variants, the xdg one
and the deprecated kde one, assume that an application that did not
negotiate any decorations will render client side decorations.

Decorations: handle results of kde-server-decoration negotiations

Decorations: minor refactoring, no logical changes intended

Decorations: respect earlier client side decoration negotion result

Before this patch, it was impossible to differentiate between negotiations
resulting in client side decorations and no negotiations at all.

By adding an enum we are now able to differentiate between the two states.

Decorations: prepare for the KDE protocol implementation

Decorations: respect earlier decoration negotiation results

Before this patch, setting `<decoration>` to `client` would cause applications
which prefer server side decorations to not have any decorations at all. This
patch fixes it by respecting the result of earlier negotiations via the
xdg-decoration-unstable-v1 protocol.

Fixes #297
Fixes #831

cursor: fix focus.followMouse glitches

Update focus correctly when using `<focus><followMouse>` and (a) the
focused window is closed; and (b) after switching desktops using action
GoToDesktop. Issue #830

Written-by: @Consolatis
Reported-by: @jech

build: bump version to 0.6.2

NEWS.md: add notes on 0.6.2

README.md: update gaming section on cursor confinement

config: enable windowSwitcher preview by default

config: change <cycleView*> to <windowSwitcher*>

Use `<windowSwitcher show="yes" preview="no" outlines="yes" />`
instead of

- `<cycleViewOSD>yes</cycleViewOSD>`
- `<cycleViewOutlines>yes</cycleViewOutlines>`
- `<cycleViewPreview>no</cycleViewPreview>`

Merge pull request #800 from Consolatis/ci/web_deploy

CI: automatically rebuild labwc.github.io

protocols: remove redundant wlr-output-management

src/config/rcxml.c: reset default values on Reconfigure

Reported-by: <umk3> via IRC

osd: add config option to disable osd

Prevent rare crash due to layering move / resize / menu operations

Fixes #817

Reported-by: @kode54

src/keyboard.c: Fix wrong argument parsing

The `wlr_keyboard.events.modifiers` signal does not use
a `wlr_keyboard_key_event` as argument, instead it is a
pointer to the `wlr_keyboard` instance which caused the
signal.

Prevent cursor based region-snapping when starting a move with A-Left

When wanting to snap to a region when starting the move
operation with A-Left (or a similar mousebind which includes a
modifier), the modifier - or another one - must be pressed again.

Fixes #761

Add FocusOutput action

Fixes #806

Suggested-by: @EysseW
Tested-by: @EysseW

src/workspaces.c: update cursor focus

Fixes #807

xdg: Reduce log priority of timeout message

xdg: Detect pending configure request timeouts

xdg: Fix visual glitch when resizing xfce4-terminal from left edge

view: Anchor right/bottom edge only when resizing via top/left edge

Currently, we anchor the right/bottom edge of the view whenever the top/
left edge is moving (current.x/y != pending.x/y). Doing so doesn't make
much sense when the right/bottom edge is also moving. In that case it's
probably best to move the view (or at least its top/left corner)
directly to its final position.

The most noticeable effect of this change is with views that don't
accept their requested size exactly when tiled or maximized (examples:
havoc, xfce4-terminal). Previously, their right-bottom corner would be
aligned with the screen edge, leaving gaps on the left and top. Now the
top-left corner will be aligned and the gaps will be on the right and
bottom. This is still not ideal, but IMHO less surprising to the user.

view: Comment updates

view: Add view_set_output()

view: Add documentation for view->output

view: Eliminate view_output() and use view->output directly

view: Make view_apply_special_geometry() return void

This makes the conditions for calling it explicit and more obvious.

view: Set output prior to view_apply_tiled_geometry()

view: Set view->output prior to calling view_center()

xdg: Set view->output prior to view_set_fullscreen()

xdg,xwayland: Set initial output for views

view: Attempt to keep non-floating views on the same output

- Make view_discover_output() static
- Call view_discover_output() only for floating views
- Deprecate view_output(); make it use view->output when possible