scaled-rect-buffer: add missing cairo_surface_flush()

buffer: remove buffer->cairo

It's more common for cairo_t to have a temporary lifetime and it will
prevent accidentally reusing its previous state.

buffer: always set buffer->{cairo,surface} for simplification

Add lsan.supp

...so suppress errors from ASAN with EGL renderer.

Also add a script to simplify running with sanitizer and this suppression
file.

Related-to: #2331

keyboard: add window rule to send release-events (#2377)

...of modifiers which are part of keybinds. This supports clients (for
example blender) that want to see modifier-release-events even if they are
part of a keybinds.

Most clients (including those using Qt and GTK) are setup to not see these
modifier release events and actually misbehave if they do.  For example
Firefox shows the menu bar if alt is pressed and then released, whereas if
only pressed (because the release is absorbed) nothing happens. So, if
Firefox saw bound modifier-release-events it would show the menu bar every
time the window-switcher is used with alt-tab.

    <windowRules>
      <windowRule identifier="blender" wantAbsorbedModifierReleaseEvents="yes"/>
    </windowRules>

Issue #1507

CI: make the smoke test start foot

CI: switch runtime tests to Arch to reduce total runtime

CI: add address sanitizer to Debian runtime tests

CI: re-enable Debian build job

CI: move comment to the right place

CI: reduce job timeout to 10 minutes

Since GitHub runners now support hardware virtualization,
the maximal runtime of the FreeBSD runner is somewhere
around 3 to 6 minutes. It may still fail sometimes so
keep the timeout parameter around.

graphic-helpers: fix segfault when medium-size icon file is loaded

lab_data_buffer->logical_{width,height} are not the actual size of the
backing buffer, so wlr_buffer.{width,height} should be used when
duplicating it in get_cairo_surface_from_lab_data_buffer().

This mistake caused segfault when an icon file 1~2 times larger than
window.button.{width,height} is loaded, because the buffer for the raw
icon file is reused by setting lab_data_buffer->logical_{width,height}
with the size for display which is different from
wlr_buffer.{width,height} (see buffer_convert_cairo_surface_for_icon()),
and it is duplicated with get_cairo_surface_from_lab_data_buffer() for
hovered/rounded variants of titlebar button.

menu: don't include menu.border.width for menu title height

It was a cruft from my experiment of adding borders around titles.

It even caused an integer overflow because theme->menu_border_width can be
INT_MIN when menu_header_height is calculated.

menu: invert the y-offset of submenus applied by menu.overlap.y

This follows Openbox's behavior.

menu: overlap submenus by menu.border.width

This follows Openbox's behavior that aligns the first item of a submenu
with its parent item

menu: support borders

This commit adds following theme configurations:

  menu.border.width: 1
  menu.border.color: #aaaaaa

theme: set default values of window.*.border.color to #aaaaaa

This makes the colors of titlebar and window borders different, but will
let menu.border.color (which will be supported soon) inherit
window.active.border.color just like Openbox does, without making the menu
borders around a selected menu item invisible.

Add scaled_rect_buffer

scaled_rect_buffer is an implementation of scaled_scene_buffer and shows
an auto-scaling bordered rectangle. This is intended for menu borders,
but can be also useful for other elements like window switcher items.

We will support rounded corners for scaled_rect_buffer in the future.

workspaces: use view_for_each_reverse() to move omnipresent views

This ensures that the view ordering and focus behavior of
omnipresent views is the same when switching workspaces.

view: add view_for_each_reverse() macro

output: fix assert() fail when enabling an output that was disabled

When the output is in the process of being enabled, but the new state
has not been committed yet, wlr_output->enabled is still false. So it's
not safe to assert that it's true in output_enable_adaptive_sync().

Fixes: 36e099fc93b3caf87e7f390a732fcec69373de8c
"view: ensure output is usable before setting adaptive sync"

docs: describe some default values

docs: align format for describing default values

scaled-font-buffer: apply buffer sharing mechanism

scaled_font_buffer.{width,height} are no longer set in _create_buffer()
since that function is not called when the buffer is reused from the
cache.

scaled-scene-buffer: implement buffer sharing mechanism

Implementers can define impl->equal() which compares two
scaled-scene-buffers so that buffers are not allocated for visually
duplicated scaled-scene-buffers.

Currently this mechanism isn't applied for scaled-font-buffers since we
haven't defined impl->equal() for it.

common/list: add WL_LIST_INIT() macro

scaled-scene-buffer: allow multiple wlr_buffer_drops()

This allows implementations of the interface to share a single
wlr_buffer for multiple wlr_scene_buffer nodes and still make
the scaled-scene-buffer handle the buffer drops.

wlr_scene_buffer holds an implicit lock while using a wlr_buffer
and scaled-scene-buffer.c holds additional locks for as long as
a buffer is within its internal cache.

This should ensure that a shared wlr_buffer never gets actually
dropped even if wlr_buffer_drop() has been called by another
scaled-scene-buffer instance using the same wlr_buffer.

NEWS.md: fix author

Merge pull request #2360 from jp7677/sfdo-app-name

feat: add libsfdo desktop entry name as OSD element

osd: add desktop entry name option

Looks desktop entry name up from libsfdo.

sfdo: add sfdo_desktop_entry_name_lookup()

icon-loader: rename to desktop-entry

Also rename `struct icon_loader` to `struct sfdo`.

src/menu.c: remove unnecessary re-renderings of font buffers

I missed that `scaled_font_buffer_update()` has `max_width` argument and
the subsequent calls to `scaled_font_buffer_set_max_width()` caused
unnecessary re-renderings.

icon-loader: fall back to app-id if icon from .desktop failed

Reported-By: LimeOn via IRC

Translation updates from weblate

Co-authored-by: Moo <hazap@hotmail.com>
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/lt/
Translation: Labwc/labwc

Remove some line terminators from log messages

rcxml.c: keep zfree(string) calls together to ease detecting missing ones

rcxml.c: fix mem leak for repeated string config entries

An example is
```xml
<theme>
  <name>Numix</name>
</theme>
<theme name="Numix" />
```

Including various other variants.
Also change all other `free(x); x = xstrdup(y)` calls to `xstrdup_replace()`.

common/mem.h: introduce xstrdup_replace()

rcxml.c: fix mem leak when deduplicating keybinds

Before this patch `keybind->keysyms` wasn't free'd when
- deduplicating keybinds
- removing keybinds due to empty action list

This patch creates a shared `keybind_destroy()` helper
which gets used in all cases where a keybind is destroyed.

theme: allow negative values for menu overlap

This fixes a regression introduced with:
https://github.com/labwc/labwc/commit/dcd9b47e5b584a2559d41cd255bb043f11970d12

menu: fix UAFs in menu_destroy() and item_destroy()

This fixes use-after-free when there's only 1 desktop and
menu_hide_submenu() is called to delete "Workspaces" submenu in
client-menu before menu scenes are initialized.

As menu_create() and item_create() no longer initialize scenes after
76515316, menu->scene_tree and item->tree should be null-checked.

placement: set default placement policy to cascade

menu: eliminate dead code

The deleted lines were dead code that didn't make any sense (even if it
were not dead, it should have recreated the parent of the hidden menu
rather than the hidden menu itself).

menu: fix menus disappearing when opening pipemenu

Commit 7651531 introduced a regression: `menu_update_scene()` which
re-creates a menu scene was called for all the menus when a pipemenu is
created, so the menus (parent of the pipemenu) were always moved to (0,0)
and hidden, and the pipemenu was incorrectly positioned.

This commit fixes it by calling `menu_update_scene()` only for the
pipemenu when it's created.

view: ensure output is usable before setting adaptive sync

Fixes #2337

menu: invalidate nested duplicated menus

Prior to this commit, nesting the same menus caused stack overflow at
`close_all_submenus()` when trying to open it.

menu: consider padding when applying menu.width.{min,max}

menu: fix "menu.title.text.justify: right" not working

menu: refactor in preparation for richer menu design

This commit separates the process of creating scene-trees of the menu
items into `{item,separator,title}_create_scene()`. This will make it
easier to draw richer menu designs like borders, paddings and rounded
corners.

src/menu.c: move some post_processing() and menu_update_width()

menu: fix UAF of server->menu_current

This fixes segfault when exiting with a menu opened, which is a regression
from eaf11fac.

workspaces: keep focus on omnipresent windows when switching workspaces

Fixes: #2329

NEWS.md: interim update

theme: rename title_height to titlebar_height

Also removed the redundant initialization of title_height in
theme_builtin().

theme: rename menu_item_padding to menu_items_padding

theme: move some variables into theme->window

For example, variables:
  `theme->window_inactive_border_color`
  `theme->window_active_border_color`
are converted to:
  `theme->window[THEME_INACTIVE].border_color`
  `theme->window[THEME_ACTIVE].border_color`

theme: reorder theme entries in documentation and codebase

server: release all the resources on exit

Release all the resources including wlr_allocator, wlr_renderer,
wlr_backend and wlr_scene. This makes it a lot easier to detect memory
leaks with ASAN.

On wlroots side, the commit ce615a44 needs to be reverted to detect memory
leaks with EGL renderer.

server: release server->ssd_hover_state on exit

cosmic-workspace: destroy manager when wl_display is destroyed

seat: destroy seat->workspace_osd_timer on exit

icon-loader: fix memory leak of icon_loader->icon_theme

osd: fix memory leak of output->osd_buffer

src/output.c: skip painting output when session is not active

On switching to a console vt when an application is painting,
labwc produces a stream of log messages of the form
```
 `00:00:52.345 [ERROR] [../src/output-state.c:45] Failed to commit frame` lines.
``

This patch tries to follow the lead from the solution to this problem
in https://github.com/WayfireWM/wayfire/pull/2484.patch

As noted in that commit message, this patch depends on unreleased
patches to seatd (master) to work reliably, and there may be a a risk
of locking up the labwc if it is used with the seatd-0.8.0.

Check that the session is initialized (is not NULL) before checking it
is active.

Co-authored-by: Madhu <enometh@net.meer>

input: add hold-begin/-end gestures

input: ensure visible cursor on gestures

input: notify idle-manager on gestures

input: use CONNECT_SIGNAL macro

environment: ignore env var assignments > 1 KiB (#2325)

...to guard against recursive constructs like FOO=$FOO:bar which would
grow on each reconfigure.

Add log message as well as a warning against this in the man page.

input: hide cursor when using touch input

Hide the cursor on touch input and keep the cursur invisible
until pointer or tablet input.

input: close xdg-popups on touch down

Taken over and now shared from the tablet implementation.

input: move notify-idle-manager to tablet/touch handlers

Move them away from the cursor emulate functions. This
avoid calling them twice for touch motion.

Also notify idle manager on touch down/up.

input: clear pointer focus on touch input

Otherwise the pointer focus could interfere with touch input, like
showing hover effect on unexpected locations.

input: move touch only with one touch point

In other words, do not move the cursor when more than
one finger is down.

input: warp cursor to touch coordinates

This makes the behavior consistent with non-touch
capable surfaces and the desktop.

meson.build: increase wlroots dep to 0.18.1

This ensures users do not run into asserts()
when using the ext-foreign-toplevel-list protocol.

foreign-toplevel: add ext-toplevel-list support

foreign-toplevel: create generic abstraction

view: add state change signals

Translation updates from weblate

Co-authored-by: Ettore Atalan <atalanttore@googlemail.com>
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/de/
Translation: Labwc/labwc

icon: matching partial strings (#2266)

...to handle for example app-id="gimp-2.10" with file "gimp.desktop"

action.c: fix style

menu: correctly center menu opened with <position {x,y}="center">

Prior to this commit, a menu opened with:

  <action name="ShowMenu">
    <position x="center" y="center" />
  </action>

was not correctly centered when menu.items.padding.x is non-zero.

Remove link to archlabslinux

Project is no longer available.

https://archlabs.github.io/

src/seat.c: Allow pointer speed of -1

Fix broken links to Openbox

Signed-off-by: tinyboxvk <13696594+tinyboxvk@users.noreply.github.com>

buf.c: fix off by one bug in buf_add_char()

Written-by: @Consolatis
Fixes: #2313

tests: add buf_add_char() test

...to prove off-by-one test with ASAN.

output: factor out configure_new_output() from new_output_notify()

... to make it possible in future to *not* immediately auto-configure
new outputs when connected. No functional change intended at this time.

output: move a few calls from new_output_notify()

... to more specific/logical locations, to make it possible in future to
*not* immediately auto-configure new outputs when connected.

In particular:

- Move regions_reconfigure_output() and session_lock_output_create()
  into add_output_to_layout(), which would be called also if the output
  is enabled later.

- Move seat_output_layout_changed() to do_output_layout_change() so it
  is called for any layout change, not just adding new outputs.

menu: use only menu_separator_padding_width to pad separators w/o label

I.e. drop the extra padding from menu_item_padding_x that got there
when separators with labels were implemented.

CI: add discussion category to IRC notification

CI: add IRC notification for discussion created

Translation updates from weblate

Co-authored-by: Roberalz <lumintnian@outlook.com>
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/es/
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/eu/
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/gl/
Translation: Labwc/labwc

tearing: avoid permanent disable due to rejected commits (#2295)

Currently, the cursor plane does not allow async page flips which causes tearing page flips to be rejected if the cursor was moved.

However, in games where no cursor image is present, the async page flips can still work as expected.

Instead of permanently disabling tearing after too many failures, test the output state first before each frame to see if we can commit with tearing_page_flip set to true.

Add github issues template

docs: add documentation for window.button.height

query: fix three-state-parameter parsing, simplify match logic

Fixes: #2288.

config/rcxml.c: fix parsing of three-state query parameters