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

Add ToggleKeybinds action

This can be used to better control Virtual Machines, VNC clients, nested
compositors or similar. All keybinds other than ToggleKeybinds itself are
disabled when first called, a 2nd call will restore handling of all keybinds.

Fixes #738
Fixes #810

xdg/xwayland: remove unnecessary assert()s

Move common xdg/xwayland handle_destroy() code to view_destroy()

view: make request_activate listener xwayland only

src/xdg.c: remove all listeners in handle_destroy()

src/cursor.c: add cursor lock confinement

src/cursor.c: Implement cursor constraints

xwayland.c: prefix view_impl functions with xwayland_view_

...to make it consistent with xdg.c

view: add move_to_front to `struct view_impl`

...to increase xwayland and xdg-shell encapsulation and to avoid passing a
function pointer as an argument in `xwayland_move_sub_views_to_front()`
which is inconsistent with labwc design patterns.

Rename view-impl.c to view-impl-common.c

Move function declarations that are common to view-implementations from
view.h into view-impl-common.h

CI: Add hwdata to FreeBSD build requirements

CI: automatically rebuild labwc.github.io

po: Add Georgian translation

po: Add Georgian translation

output: Add output_nearest_to()

Reimplement output_from_cursor_coords() as output_nearest_to_cursor().

xdg: Use wlr_xdg_surface_get_geometry() to get size

This fixes an issue with havoc not having a valid size on map().

Investigation showed that xdg_surface->current.geometry is set only by
the xdg_surface::set_geometry protocol message, which is optional. If
set_geometry is not called, then we are supposed to compute the size
from the surface buffer(s). wlr_xdg_surface_get_geometry() already
accounts for this, so we just need to use wlr_xdg_surface_get_geometry()
instead of reading xdg_surface->current.geometry directly.

xdg: Remove unused #include

README.md: suggest using nested gamescope for gaming

Fixes #690

view: Rediscover output in view_adjust_for_layout_change()

view: Handle fullscreen also in view_apply_special_geometry()

view: Factor out set_fullscreen() helper from view_set_fullscreen()

xdg: Save natural_geometry.x/y with initially maximized view

Fixes an issue where, if Thunar was started maximized, it would
un-maximize to the top-left corner rather than the center.

xdg: Use view_center() for parent-relative centering

- Add optional struct wlr_box* ref argument to view_center()
- Perform margin adjustment within view_center()
- Expose view_output() to fetch parent's output

view: Align large views to usable area

view: Add optional output parameter to view_center()

Allows centering the view on a specific output without the workaround of
overwriting view->current.x/y.

labwc-actions(5): update action format

Revert "xwayland: Fix size issue when starting VLC fullscreen"

The fix caused a couple of issues:

1. Ignoring client configure requests caused some clients to hang
   and not repaint correctly. We are supposed to synthesize a
   ConfigureNotify event when ignore/override a client configure
   request, but this isn't possible with current wlroots.

2. Setting view->natural_geometry from client configure requests
   resulted in overwriting good values with bad in some cases (e.g.
   with tiled xfce4-terminal in xwayland mode).

For now, revert the fix. This does allow clients to mess with view
positioning for maximized/fullscreen/tiled views, but right now the
alternatives seem worse.

The original specific issue (VLC undoing its fullscreen geometry)
is arguably a bug in VLC anyway.

This reverts commit 09599861ac726b142a79ed60035809f04c151457.

view: Tidy up view->output/view->fullscreen redundancy

A fullscreen view currently has its output specified twice by:
  - struct output *output
  - struct wlr_output *fullscreen

view->fullscreen may also become a dangling pointer if the output is
disconnected, because view_on_output_destroy() clears view->output but
not view->fullscreen.

To eliminate the redundancy and the dangling pointer, let's change
view->fullscreen to a Boolean and rely on view->output to specify the
output.

Along the way, change a few related usages of struct wlr_output to
struct output as well.

No functional change intended.

v2: Don't allow entering fullscreen on disabled output (makes
    conditions for entering/leaving fullscreen symmetric)
v3: Use output_is_usable() helper

output: Add output_is_usable() helper

region: Add output pointer to struct region

region: Add #include to make header dependency explicit

view: Add view_evacuate_region() helper

v2: Modify asserts as suggested by Consolatis

view: Unify view_move()/view_move_resize()

view->impl->move() is a specific case of view->impl->configure().
To reduce code duplication, we can use view->impl->configure() for
pure moves (without resize) as well.

xwayland's move() function also possibly contained a race condition
when there was a pending resize, as it used the current surface
width/height rather than the pending width/height. This is fixed.

desktop: Avoid centering views without initial geometry

ssd: Fix segfault when disabling output with fullscreen view

wlr_output_layout_get() returns NULL for disabled outputs.

Backtrace (abbreviated):

     #0  lab_wlr_output_layout_layout_coords at ../src/ssd/ssd_extents.c:33
     #1  ssd_extents_update at ../src/ssd/ssd_extents.c:133
     #2  ssd_extents_create at ../src/ssd/ssd_extents.c:95
     #3  ssd_create at ../src/ssd/ssd.c:161
     #4  decorate.part.0.lto_priv.0 at ../src/view.c:637
     #5  decorate at ../src/view.c:636
     #6  view_set_fullscreen at ../src/view.c:716
     #7  view_adjust_for_layout_change ../src/view.c:745
     #8  desktop_arrange_all_views ../src/desktop.c:52
     #9  output_update_all_usable_areas at ../src/output.c:495
     #10 output_update_for_layout_change at ../src/output.c:263
     #11 do_output_layout_change at ../src/output.c:423
     #12 do_output_layout_change at ../src/cursor.c:267
     #13 output_config_apply at ../src/output.c:334
     #14 handle_output_manager_apply at ../src/output.c:354

src/xdg.c: sync move with pending resize

Before this patch, configuring a surface with a new size,
immediately followed up by one or more view_move() calls
would move the surface to the new coordinates immediately
without waiting for the resize. This caused visual glitches
when for example dragging a maximized window: the position
would change but the size was still that of a maximized
window.

This patch fixes that by just ignoring view_move() requests
(but still updating view->pending) if there is a configure
request pending. Once the client commit comes in the new
size will be applied as usual.