desktop: work around client-side rounding issues at right/bottom pixel

This also avoids a similar server-side rounding issue with some
combinations of wlroots and libwayland versions. See:

- https://gitlab.freedesktop.org/wayland/wayland/-/issues/555
- https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/5159

Tested with qmpanel (similar to lxqt-panel) at 1x and 2x output scale.
Does not help if scaling is done client-side, e.g. QT_SCALE_FACTOR=2.

Fixes: #2379
Fixes: #3099

labnag: use overlay layer by default

...so that the dialog is still visible when some client is using
fullscreen mode.

NEWS.md: interim update

scripts/check: run style check for test files

Co-authored-by: @johanmalm

CI: also run on t/ changes

tests: remove placeholders for arguments

Don't remove newlines when parsing config, menu and XBM

Removing newlines in rc.xml and menu.xml caused parser error with
following content:

<!--
 -
 - Some comments
 -
-->

...though it is a valid XML.

Let's not do that. I moved `grab_file()` to `buf.c` and renamed it to
`buf_from_file()`, because it now directly touches `struct buf` and
I don't like having a source file only for one function.

Translation updates from weblate

Co-authored-by: alvaroelpob <alvaropobladoresteban9@gmail.com>
Co-authored-by: p-bo <pavel.borecki@gmail.com>
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/ca/
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/cs/
Translation: Labwc/labwc

theme: replace THEME_(IN)ACTIVE with SSD_(IN)ACTIVE

windowswitcher: show 's' as "state" for shaded views

While at it sorted the code to show 'm' before 's' and 's' before 'M'
- from the least visible to the most visible state.

implement is_modal_dialog() for xdg shell

rcxml: call labnag with --keyboard-focus on-demand by default

labnag: add --keyboard-focus option

The new `--keyboard-focus [none|on-demand|exclusive]` option (default:
`none`) allows to some keyboard controls in labnag:

  Right-arrow or Tab: move the button selection to the right
  Left-arrow or Shift-Tab: move the button selection to the left
  Enter: press the selected button
  Escape: close labnag

The selected button is highlighted with the inner 1px border. Maybe we can
instead use different colors for the selected button, but I prefer the
inner border for now because it doesn't require us to add new color
options or make them inherit labwc's theme.

labnag: remove redundant lines in conf_init()

labnag: s/LAB_EXIT_TIMEOUT/LAB_EXIT_CANCELLED/

osd: allow window switcher to temporary unshade windows

This can be configured with a new unshade="yes|no"
argument for windowSwitcher in rc.xml

Fixes: #3111

view: store title/app_id in view

This simplifies our codes and eliminates duplicated
`view.events.new_{title,app_id}` events. This should not change any
behaviors.

osd-thumbnail: make sure item->{normal,active}_title are non-null

The if-statement doesn't make sense, because `view_get_string_prop()`
never returns NULL. And if it did, it would cause segfault in
`osd_thumbnail_update()`.

menu: remove redundant cast

osd,ssd: don't cast away const

menu: use xmlFree() for return value of xmlGetProp()

osd-classic: add theme options for selected window item

This commit adds new theme options:
- osd.window-switcher.style-classic.item.active.border.color
- osd.window-switcher.style-classic.item.active.bg.color

These theme options configures the border/background of selected window
item in the `classic` style window switcher. Their default values are
identical to `thumbnail` style window switcher, which means the default
border color is now `osd.label.text.color` with 50% opacity and the
default background color is now `osd.label.text.color` with 15% opacity.

action: simplify action_prompt_command()

build: fix build with libinput as a subproject

Unfortunately, has_header_symbol() doesn't work with internal dependencies.

Ref: https://github.com/mesonbuild/meson/issues/13553

build: bump version to 0.9.2

view: fix unexpected view->tiled with SnapToEdge against centered view

In 2ac4811, I was missing that windows can be tiled to "center".
As a result, after executing
`<action name="SnapToEdge" combined="yes" direction="left" />` against a
center-tiled window, `view->tiled` is set to `CENTER|LEFT`.

NEWS.md: update notes for 0.9.2

rcxml: move <maximizedDecoration> from <core> to <theme>

fix typo in comment

fix double free for libxml2 < 2.13

xmlAddChild() only unlinks the second argument since libxml2 2.13.

regression from 503af105

desktop-entry: fix wrong description of sfdo-icon flags

CI: also run on clients/ changes

build: use spaces instead of tab

desktop-entry.c: don't demote error messages with LABWC_DEBUG_LIBSFDO

Also add additional logging to tell users how to get
more information about failures to load the icon theme.

desktop-entry.c: on detecting a broken icon theme, fall back to hicolor

Fixes: #3126
Reported-By: Kreevoz

osd-thumbnail: update default colors of selected window item

Previously, the default values of
`osd.window-switcher.style-thumbnail.item.active.{bg,border}.color`
were blue. But they caused the selected window title in the window
switcher to be unreadable due to duplicated colors of the text and
background with Openbox themes like Numix.

Instead, this commit updates them to follow other themes configurations.
The default border color of the selected window item is now
`osd.label.text.color` with 50% opacity and the background is
`osd.label.text.color` with 15% opacity.

For subpixel antialiasing to work, the background color is calculated by
manually blending `osd.label.text.color` and `osd.bg.color`, rather than
just updating the alpha with 50% or 15%.

src/common/graphic-helpers.c: fix declaration/definition arg name difference

include/common/scene-helpers: fix declaration/definition arg name difference

include/common/buf.h: fix declaration/definition arg name difference

include/common/box.h: fix declaration/definition arg name difference

labnag: rename options for color

include/common/xml.h: fix declaration/definition arg name difference

Privatize view_append_children()

Privatize desktop_topmost_focusable_view()

Privatize view_contains_window_type()

Privatize output_manager_init()

Privatize xwayland_surface_from_view()

Privatize rcxml_parse_xml()

Privatize private view_get_root()

build: add 'sections' option to show unused functions

CI: use libwlroots-0.19-dev build-dep for Debian

CI: add unit tests

test/xml: use xmlBufferContent()

xmlBuffer->content has been deprecated.

test: fix build error in t/xml.c

38a1a9b broke `t/xml.c` due to `macros.h` requiring `wlr/version.h`.
This commit fixes it by adding `wlroots` as a direct dependency of the
test executables.

action: fix UAF when using prompt during reconfigure

Reported-by: @jlindgren90
Fixes #3106

CI: fix broken FreeBSD CI by setting -Dlibsfdo:b_ndebug=false

...because with with

    meson setup build -Dbuildtype=release -Db_ndebug=true \
        --werror --force-fallback-for=libsfdo

we get the following warning:

    In file included from ../subprojects/libsfdo/common/dirs.c:5:
    ../subprojects/libsfdo/include/common/membuild.h: In function ‘sfdo_membuild_validate’:
    ../subprojects/libsfdo/include/common/membuild.h:29:65: error: unused parameter ‘membuild’ [-Werror=unused-parameter]
       29 | static inline void sfdo_membuild_validate(struct sfdo_membuild *membuild) {

...because `sfdo_membuild_validate()` contains nothing but an `assert()`
and that therefore results in an `unused-parameter` warning with `NDEBUG`.

https://gitlab.freedesktop.org/vyivel/libsfdo/-/blob/main/include/common/membuild.h?ref_type=heads#L30

Add translate.h for HAVE_NLS includes/defines

...to shrink labwc.h footprint

Remove unused function key_state_nr_pressed_keys()

Remove unused function lab_xml_get_node()

Remove unused function trim_last_field()

Remove unused function node_layer_popup_from_node()

Remove unused function output_max_scale()

Remove unused function scaled_font_buffer_set_max_width()

Remove unused function menu_call_actions()

rcxml.h: minor tweaks to order of variables

labwc-config(5): document <promptCommand>

config: add `<core><promptCommand>`

...to enable configuration of the action prompt command.

Also set some better defaults for labnag.

The new default command is:

    labnag \
        --message '%m' \
        --button-dismiss '%n' \
        --button-dismiss '%y' \
        --background '%b' \
        --text '%t' \
        --border '%t' \
        --border-bottom '%t' \
        --button-background '%b' \
        --button-text '%t' \
        --border-bottom-size 1 \
        --button-border-size 3 \
        --timeout 0

...where the conversion specifiers are defined as follows:

    %m: the `<prompt>` message option
    %n: _("No")
    %y: _("Yes")
    %b: osd.bg.color
    %t: osd.label.text.color

This config options also enables the use of a different dialog client, for
example like this:

    <core>
      <promptCommand>zenity --question --text="%m"</promptCommand>
    </core>

labnag: fix segfault caused by providing --timeout as long option

font: remove 4px padding on the right

Added `menu.items.padding.x` padding between item text and arrow instead.

Replaced `if (!string)` with `if (string_null_or_empty(string))` in
`font_extents()` just as a minor optimization.

README.md: remove high-level scope summary

...as it is very old and not relevant anymore.

default-bindings.h: set combine="yes" for SnapToEdge keybinds

action: allow SnapToEdge to combine two cardinal directions

This patch adds `combine` argument to (Toggle)SnapToEdge actions.
This allows to snap a window to e.g. up-left by running two actions:
- `<action name="SnapToEdge" direction="left" combine="yes" />`
- `<action name="SnapToEdge" direction="up" combine="yes" />`

Then running `<action name="SnapToEdge" direction="down" combine="yes" />`
snaps it to left again. This behavior is almost the same as KWin, except
that snapping a up-right-tiled window to right doesn't move it to the
right-adjacent output, but makes it right-tiled first.

view: remove an obsolete code in view_snap_to_edge()

We no longer need to call view_apply_tiled_geometry() there, since we now
clear view->tiled when dragging a tiled window since 9f51384.

libsfdo.wrap: update revision to v0.1.4

README.md: remove yambar reference as discontinued

Update labwc-actions.5.scd

NEWS.md: interim update

view: fix <query monitor="current|left|right" />

Before this commit, <else> branch was always executed with
monitor="current", monitor="left" or monitor="right" queries.

For example:

<action name="If">
  <query monitor="current" />
  <then>
    <action />
  </then>
  <else>
    <action />
  </else>
</action>

clang-format: tweak to match existing code a little better

"clang-format -i src/view.c" before:
  1 file changed, 204 insertions(+), 169 deletions(-)

"clang-format -i src/view.c" after:
  1 file changed, 181 insertions(+), 146 deletions(-)

rcxml: use const char* for string literals

osd-thumbnail: put designated initializers in order

interactive: add braces around case containing declaration

rcxml: use fixed arrays for rc.title_buttons_*

These are just lists of enum lab_node_type, with a bounded size and
no middle-insertions/removals, so linked lists are overkill.

Also, the use of wl_list_for_each[_reverse] just to access the first or
last entry in the list (corner button) was weird.

CONTRIBUTING.md: fix some typos, lots of Oxford commas

The primary change is “Github”/“github” ⇾ “GitHub”, but there are plenty of others.

labwc-config(5): add example for autoEnableOutputs

...with inspiration from example in #3059 by @jlindgren90

desktop: refactor get_cursor_context()

This patch should not change any behaviors.

This clarifies the semantics of cursor_context returned by
get_cursor_context() as I described in cursor.h; when cursor is on a
subsurface (e.g. xdg/x11/layer/session-lock), the returned ctx.surface
and ctx.node points to the subsurface rather than its parent.

desktop: let get_cursor_context() return layer-popup subsurface

With this change, when a layer-popup has a subsurface and cursor is on
it, wl_pointer.enter/motion events are sent to the subsurface rather
than its parent layer-popup surface. I think this follows wayland spec
more closely.

Tested with my demo: https://github.com/tokyo4j/wayland-demo/tree/layer-popup

desktop: don't use LAB_NODE_LAYER_SUBSURFACE node type

Instead, set ctx.type = LAB_NODE_LAYER_SURFACE for both layer-surfaces
and layer-subsurfaces.

This patch preserves the existing behaviors:
- Pressing a subsurface of an on-demand layer-surface gives pointer
  focus to the subsurface, but gives keyboard focus to the parent
  layer-surface (related: a5fcbfaf).
- Pressing a subsurface of a layer-surface doesn't close a popup
  (related: a89bcc3c).

ssd: attach LAB_NODE_SSD_ROOT to ssd->tree

This doesn't change any behaviors.

Attaching LAB_NODE_NONE node-descriptor to ssd->tree looks strange, this
patch uses new LAB_NODE_SSD_ROOT instead. The node-descriptor attached to
ssd->tree is needed for get_cursor_context() to detect cursor hovering on
borders/extents.

I also updated get_cursor_context() to make my intent clearer.

output: remove node descriptors

scaled-buffer: remove *_from_node() functions

desktop.c: don't print errors when cursor is on resize-indicator

In f347a81, I thought there are only window contents and SSD under
`view->scene_tree` and forgot about the resize-indicator.

I also refactored the logic around it:
- Remove `ret.node->type == WLR_SCENE_NODE_BUFFER` check since it's
  already done by `lab_wlr_surface_from_node()`
- Eliminate duplicated call to `lab_wlr_surface_from_node()`

src/output.c: restore drm lease include

Introduced in #3062.

Its usage is guarded by a wlroots version check
which prevented the CI to detect the issue.

When compiled with a wlroots version > 0.19.0
(like the chase 0.20 PR) the error shows up.

tree-wide: use forward declarations for wlr types

src: remove unused #includes (via include-what-you-use)

include: reduce unnecessary #includes

array: use die_if_null() from common/mem.c

idle_manager: remove unused parameter and field

wlr_seat isn't used anywhere.

Translation updates from weblate

Co-authored-by: virtual-hand <handvirtual@tutamail.com>
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/pt_BR/
Translation: Labwc/labwc

ssd: update comments referencing ssd_part/ssd_part_type