Support png buttons

In the theme directory add close-{active,inactive}.png instead of
close.xbm - and similarly for iconify, menu and max.

theme: simplify loading of xbm buttons

No functional change.

src/cursor.c: end popup grab on desktop click

Fixes #949

Restore SnapToEdge and MoveToEdge default keybinds

When adding the fallback keybinds we add them as string but expect them being an int.
This commit fixes that by using the same parsing routines that are used when parsing
user supplied configuration.

Fixes 1ee8715d57ddb6b444e0b089879db6f837400539
actions: use enum for _ToEdge action

Fix crash when using _ToEdge actions when using the default keybinds

This happens because of two separate bugs:
- The action validation failed to verify the data type of the argument
- When adding the fallback keybinds we add them as string but expect them being an int

This commit fixes the first bug.

Fixes 1ee8715d57ddb6b444e0b089879db6f837400539
actions: use enum for _ToEdge actions

view: make move_sub_views() use append_children method

...to share common code with minimize_sub_views()

Also, fix a bug in the move-to-back functions to move the window
hierarchy in the right order.

Helped-by: @Consolatis

view: minimize parents/children together

Minimize the whole view-hierarchy from top to bottom regardless of which
one in the hierarchy requested the minimize. For example, if an 'About' or
'Open File' dialog is minimized, its toplevel is minimized also, and vice
versa.

For reference:
- This is consistent with in openbox, where child views (dialogs) can be
  minimized, but when doing so the parent is also minimized.
- In mutter these types of dialogs cannot be minimized (via client-menu or
  otherwise).
- In both openbox and mutter, when a toplevel window is minimized any open
  children are also minimized.

xwayland: fix client request-unmap bug

Remove foreign-toplevel handle when a client sends a request-unmap to a
minimized (therefore unmapped) view.

xwayland: guard against mapping view without surface

...which may occur if a user minimizes an xwayland view (typically a
child view such as a dialog) at the same time as the client sends a
request-unmap, which xwayland clients sometimes do without actually
requesting destroy and just leave them dangling.

view: move (z-order) ancestors with modal dialogs

...so that other window cannot be positioned between modal dialogs and
their parent windows. This is consistent with Gtk3 and Qt5 applications on
mutter and openbox.

xdg: show modal dialog above parent window

This behaviour is consistent with that of mutter and openbox.

Fixes: issue #823

labwc-theme(5): describe color alpha value

...which has been supported for a long time but not been reflected in the
man page.  Colors can be parsed as `#rrggbb aaa` with aaa representing a
decimal alpha value. This could be used to hide a button, for example:

    window.active.button.menu.unpressed.image.color: #000000 0
    window.inactive.button.menu.unpressed.image.color: #000000 0

actions: use enum for _ToEdge actions

This also improves the config robustness as invalid edge names will now
prevent the action to be created in the first place and the user gets
notified about the issue.

view: move MoveRelative code from action.c and restore natural geometry

keyboard: fallback on raw keysyms for bindings

When looking up keybinds, if the translated keysyms (based on the keymap
for the keyboard) do not match a defined keybind, try raw keysyms (as if
there were no modifier translation).

This allows a user to define for example keybind with "S-1" rather than
"S-exclam". It also supports "W-S-Tab".

Fixes: issues #163 #365 #992

keyboard: combine nsyms and *syms in struct

...in preparation for handling raw keysyms

No functional change intended.

theme: limit corner radius to the height of the titlebar

theme: fix rounded-corner bug

Make top left/right corner pieces with large border thickness pixel
perfect.

Fixes: issue #988

Chase wlroots: Stop using wlr_xwayland_surface event data

Chases: 27edd024f83892f4af9c5084d47b73f26966aaf1
xwayland: pass NULL as event data

Chase wlroots: Add missing #include

Chases: 323f0b94db22877487bca073fca497cd033719e8
wlr_scene: Cleanup header includes

xwayland: Fix segfault at exit (seen with wlroots master)

Added missing closing tags

build: bump version to 0.6.4

NEWS.md: add notes on 0.6.4

dnd: fix focus at end of dnd operation

By default, leave keyboard-focus with the surface that originally had it
at the end of a dnd operation.  This is consistent with the default
behaviour of openbox and mutter.

If the 'focus/followMouse' option is enabled, then focus on the surface
under the cursor at the end of the dnd operation.

Fixes: issue #976

Add support for ext_idle_notify_v1..

..and combine both idle variants into a standalone manager

find-banned: support reading multiple files from stdin

find-banned: print filename with each unwanted token

scripts/helper/find-idents.c: handle single line comment tokens

Add scripts/find-banned.sh

ToggleKeybinds per window

osd: move get_osd_height() closer to where it is called from

osd: make item-border part of item-height

theme: add option osd.window-switcher.item.active.border.width

...and change default values for the variables below to keep the
window-switcher look the same as it was at the last release.

osd.window-switcher.padding = 4
osd.window-switcher.item.padding.y = 1
osd.window-switcher.item.active.border.width = 2

theme: add option osd.window-switcher.padding

Fixes: issue #969

src/action.c: relax argument checks when running actions

config: validate keybind, mousebind and menu actions

src/action.c: add action_is_valid()

src/action.c: enforce action arg->key being set

action.c: remove duplicate code

menu: support CDATA for <command> element

...in order to support obmenu-generator output such as this:

  <menu id="root-menu" label="Applications">
    <item label="File Manager"><action name="Execute"><command><![CDATA[xdg-open .]]></command></action></item>
    <item label="Terminal"><action name="Execute"><command><![CDATA[xterm]]></command></action></item>
    <item label="Web Browser"><action name="Execute"><command><![CDATA[xdg-open http://]]></command></action></item>
    <item label="Run command"><action name="Execute"><command><![CDATA[gmrun]]></command></action></item>
  </menu>

References:
- https://github.com/trizen/obmenu-generator
- https://aur.archlinux.org/packages/obmenu-generator
- https://trizenx.blogspot.com/2012/02/obmenu-generator.html

Fixes: issue #972

checkpatch.pl: allow xmlFree in CamelCase format

...to avoid 'Avoid CamelCase' warning.

checkpatch.pl: remove 'spaces preferred around that...' warning

...because it gives a false positive for "xmlChar *foo" saying it should
be "xmlChar * foo".

menu: print menu.xml nodenames

...when LABWC_DEBUG_MENU_NODENAMES is set

action: disable MoveRelative for fullscreen and unmaximize

view: resize_relative disable for fullscreen and maximized and set untiled

view: add client_request flag to view->impl->unmap()

This makes explicit the subtle behavioral difference between
xwayland_view_unmap() and handle_unmap().

With this change, the XDG and XWayland versions of handle_map/unmap()
are now identical, which will make further refactoring possible.

osd: use theme->osd_border_width for focused item

theme: add window-switcher width/padding

...and calculate item-height based on font-height

Add theme options
  - osd.window-switcher.width
  - osd.window-switcher.item.padding.x
  - osd.window-switcher.item.padding.y

Issue #879

osd: use theme->osd_border_width consistently

Some formatting

action: add ResizeRelative

add MoveRelative to man page and use view->pending

add MoveRelative

session.c: unconstify build_path() return value

common/string-helpers.c: add strdup_printf()

window-rules: support matchOnce attribute

...allowing a rule to be applied to only the first window matching a
particular criteria. For example, the following can be used to apply a
window rule to lxqt-panel but not its configuration window with the same
app_id:

    <windowRules>
      <windowRule identifier="lxqt-panel" matchOnce="true">
        <skipTaskbar>yes</skipTaskbar>
        <action name="MoveTo" x="0" y="0" />
        <action name="ToggleAlwaysOnTop"/>
      </windowRule>
    </windowRules>

action: add MoveTo

action: support arguments of int type

config: support <margin top="" bottom="" left="" right="" output="" />

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