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

query: support additional conditions for matching clients

Co-authored-by: Andrew J. Hesford <ajh@sideband.org>
Closes: #2245.

session-lock: make session_lock_output_create() safe to call twice

session_lock_output_create() can safely no-op if the lock output has
already been created for the specified output. This scenario doesn't
happen currently, and the change is in preparation for some other
output-related changes I am working on. But I think it's a nice code
improvement worth merging separately.

output: remove ported wlr_output functions

We have several wlr_output_* functions which are just wrappers around
corresponding wlr_output_state_* functions and don't actually touch the
wlr_output itself. These probably made some sense historically, but IMHO
they are just confusing now. So remove them and call wlr_output_state_*
directly.

Rename wlr_output_commit() (still useful) to output_state_commit().

Add mouse emulation for touch devices (#2277)

src/main.c: add environment variable LABWC_VER
src/config/session.c add env var LABWC_VER
This could be useful going forward for helper programs
such as `labwc-menu-generator` or `labwc-tweaks`

keyboard: broadcast modifiers

...to all clients rather than just the one with keyboard focus on keyboard
enter/create, modifer press/release and wlr_seat_set_keyboard().

This enables:

- Clients such as panels to display the current keyboard layout without
  introducing new wayland protocols or other IPC.

- Unfocused xdg-shell clients to understand button press with keyboard
  modifiers for example Ctrl+click.

The keymap is forwarded to all clients in wlr_seat_set_keyboard(). When a
keymap contains multiple layouts, the selection is made via modifiers,
which previously were only sent to the client with keyboard focus.

Tested with: https://github.com/johanmalm/keyboard-layout

Ref: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4496

Fixes: #2271

menu: use "MenuHeader" font height in separators with label

Translation updates from weblate

Co-authored-by: Peter Jespersen <flywheel@illogical.dk>
Translate-URL: https://translate.lxqt-project.org/projects/labwc/labwc/da/
Translation: Labwc/labwc

build: bump version to 0.8.1

NEWS.md: update for 0.8.1

build: require libsfdo >=0.1.3

menu: fix UAF in client-list-combined-menu after window destruction

Update client-list-combined-menu when a window is destroyed to avoid
SEGFAULT when selecting an that window entry in it.

docs: replace two 'we's with passive forms

To match the common style in docs/

src/icon-loader.c: skip non application entries for wmclass lookup

Fixes: #2254

input: add comments about tablet mouse emulation