NEWS.md: update notes on 0.6.0

ci: add hwdata

wlroots.wrap: use 0.16.0

Refuse to start when SUID is detected

This ensures that those surprised by the deprecation of SUID operation
receive an error rather than accidentally having run as root.

swaywm/sway@e572805

update repeat rate of all keyboard input devices, not keyboard group's keyboard

without this patch, reconfiguring keyboard repeat rate/delay at runtime does not work

cursor: combine two variables relating to frame-context

Mouse bindings, unlike key bindings, are made within contexts which
represent what was clicked/dragged.  The context 'Frame' refers to the
entire window frame including both the window decorations (if any) and the
client window itself. It is typically used for alti + left/right click to
move/resize the window.

'Frame' is a special case in that when a button is bound in this
context, the action will not be forwarded to the client, which is what
we describe with the 'consumed_by_frame_context' variable.

output: test adaptive sync

Required by https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3637
See also swaywm/sway@07bfeb2

Fixes #516

Support smooth scroll and horizontal scroll

note that this changes Scroll mousebinds from taking a "button"
attribute to taking a "direction" attribute

cursor: Make update_pressed_surface() no-op except during grabs

This matches Weston behavior more closely and fixes Firefox context
menus.

view: survive setting fullscreen with no outputs

Fixes #608

src/config/rcxml.c: distinguish no and unknown font places

Currently, the `rc.xml` parser applies font settings in a `<font>` tag with an
unknown value for its `place` attribute to all fonts. This means that whatever
the final unknown-`place` `<font>` tag is in a user's `rc.xml` applies to all
text drawn by labwc.

Instead, only treat `<font>` tags with an empty or missing `place` attribute as
applying globally, and warn when encountering unknown `place` attribute values
(which will help us find font places to support).

keyboard: use 1000/rate msecs for keybind repeat event source delay

fix typo

src/cursor.c: Use 'handled' for state of consumed cursor actions

Scroll-Bindings: Add docs

Scroll-Bindings: Add default config

By default we switch workspaces when scrolling
on the desktop to match the Openbox behavior.

Scroll-Bindings: Add support for mouse wheel bindings

Fix coding style

checkpatch: ignore += UNNECESSARY_ELSE

checkpatch: add scripts/check for batch processing

Co-Authored-By: @Consolatis

checkpatch: patch to suit labwc coding style

Coding style specific changes include:

- Accept pango, glib and libxml2 CamelCase variables
- Remove "need consistent spacing around '*'" warning
- Do not warn about "structs that should be const"
- Do not warn on braces {} around single statements
- Do not warn about braces {} for single statement blocks
- Do not warn about wanting { on previous line for functions
- Remove check for missing blank line after declaration
- Ignore SPLIT_STRING COMPLEX_MACRO PREFER_KERNEL_TYPES
  LOGICAL_CONTINUATIONS PARENTHESIS_ALIGNMENT OPEN_ENDED_LINE
  MACRO_ARG_REUSE PREFER_FALLTHROUGH ARRAY_SIZE INITIALISED_STATIC

Other changes include:

- Do not complain about missing spelling.txt
- Print filename for each file, but no summary line

checkpatch: import checkpatch.pl from Linux kernel

https://raw.githubusercontent.com/torvalds/linux/4ce9f970457899defdf68e26e0502c7245002eb3/scripts/checkpatch.pl

Chase wlroots: output test assert

This fixes an assert on output test when
running with the headless backend.

To update the wlroots subproject use
meson subprojects update wlroots

Chases wlroots 05454618cd2d49fb3a5f0c560b0d2c455cf32467
xwayland: split headers

Fixes #605

keyboard: Implement key repeat for keybindings

It seems that every Wayland client is expected to implement its own
key-repeat logic, rather than doing it server-side as in X11.  This
means that labwc also has to implement its own key-repeat logic for
compositor keybindings.

This is a very simplistic timer-based implementation.  It doesn't
attempt to synthesize accurate timestamps, and may lag depending
on system load, but it appears to get the job done.

v2: Use server->wl_event_loop
v3: Comments and formatting

key-state: Prevent array overflow

- Prevent adding the same keycode more than once
- Prevent adding more keycodes than MAX_PRESSED_KEYS

src/menu/menu.c: Add comments

README: add link to pre-0.6.0 video

rcxml.c: combine common code in fill_{key,mouse}bind()

rcxml.c: support 'to.action' in mousebind

...in support to specifying 'left', 'right' and so on with actions
GoToDesktop and SendToDesktop.

For example:

    <mouse>
      <default/>
      <context name="Desktop">
        <mousebind button="Up" action="Scroll">
          <action name="GoToDesktop" to="left"/>
        </mousebind>
        <mousebind button="Down" action="Scroll">
          <action name="GoToDesktop" to="right"/>
        </mousebind>
      </context>
    </mouse>

xwayland: Hide unmanaged nodes on unmap()

Since xwayland commit 9e4b7948df5a9a67632bb606150ec171b5519672
there is a one-second delay between XWayland surfaces being
unmapped and destroyed.  This revealed that we were not actually
handling the unmap() signal correctly for unmanaged surfaces.

cursor: Restore drag icon after the move to scene-graph

Also move everything DnD related to src/dnd.c

Add --exit and --reconfigure

Add long command line options

server: do not set WAYLAND_DISPLAY twice

labwc(1): describe SIGTERM and SIGHUP

Set environment variable LABWC_PID

...to the pid of the compositor so that SIGHUP and SIGTERM can be sent to
specific instances using `kill -s <signal> <pid>` rather than
`killall -s <signal> labwc`

Issue #573

build: simplify protocol paths

No need for arrays here.

swaywm/sway@04f8a65

cursor: Call wlr_seat_pointer_notify_clear_focus() only if needed

@Consolatis determined that apparently the extra "clear focus"
event confuses the drag source during drag-and-drop.

Fixes drag-and-drop of files into folders within the same Thunar
window.

Chase wlroots: unreadable displays

This fixes an issue of unreadable displays in some
multiple output configurations.

To update the wlroots subproject use
meson subprojects update wlroots

Chases wlroots 694e9bbb9d7114f39311d93e885e010606a88dae
backend/drm: allocate connector CRTC on lease creation

Fixes #583

CONTRIBUTING.md: describe coding style more accurately

- Clarify the coding style relative to Linux kernel and Devault rules
- Add g_pattern_match_simple() example
- Add namespace advice and preference
- Elaborate on preferred header documentation style.
- Add table of contents
- Simplify API section and add wl_list_append()

s/wl_list_insert(list.prev...)/wl_list_append(list...)/

common: add wl_list_append()

NEWS.md: update notes on 0.6.0

CI: Switch Void mirror to repo-ci.voidlinux.org

Chase wlroots: output cursor

To update the wlroots subproject use
meson subprojects update wlroots

Chases wlroots 68c8cef38edafa5e52532e0b43a4888554168361
cursor: Don't warp to (0,0) when last output is disconnected

Fixes #561

CI: FreeBSD switch from 'quarterly' to 'latest'

Latest wlroots requires libdrm-2.4.113 but 'quarterly' only
provides libdrm-2.4.112. Switch to 'latest' to fix this.

src/osd.c: Ensure destroying views are not used as preview anchors

src/osd.c: Prevent showing invalid windows

This might happen when closing the last
application "to tray" like VLC, Discord or Steam.

Reported-by: @Flrian

keyboard: put modifier/key listeners in keyboard struct

...so that it can be determined what wlr_keyboard events come from.

This is required to manage virtual keyboards alongside the keyboard_group
of physical keyboards.

seat: implement virtual keyboard protocol

seat: implement virtual pointer protocol

seat: add seat_add_device helper

seat: add seat_update_capabilities helper

view: no gap for top/left align big window

On initial positioning of toplevel windows we call view_center().
During the centering process, if it turns out that the view is larger than
the output usable-area then we just top/left align it with no gap.

Relates to the gap aspect of issue #403

Reported-by: @Flrian

CI: Add Debian testing

Fix minor coding-style violations

...based on https://github.com/johanmalm/checkpatch.pl

CONTRIBUTING.md: improve all sections

...including guidelines on:

- the use of glib
- the use of GNU C extensions
- what commit messages should look like
- coding-style

cursor: Prevent setting cursor icon on drag

Partly fixes #549

Merge pull request #438 from johanmalm/fix/repeat

Revert "keyboard: cancel repeat when handling key-bind" and
only pass on pressed+sent keycodes to surfaces on keyboard-focus.

seat: only pass on sent keys on surface-focus

Key events associated with keybindings (both pressed and released) are not
sent to clients. When using wlr_seat_keyboard_notify_enter() it it
therefore important not to send the keycodes of _all_ pressed keys, but
only those that were actually _sent_ to clients (that is, those that were
not bound).

This approach is consistent with sway's implementation in input/seat.c
https://github.com/swaywm/sway/blob/cffb006feba52c318e66f73c3463032fa76782dc/sway/input/seat.c#L173-L175

Fixes issue #510

src/keyboard.c: register keys before inhibit check

Call key_state_set_pressed() before checking
seat->active_client_while_inhibited to avoid missing release events for
clients using the inhibit protocol (for example swaylock).

src/keyboard.c: do not end window-cycling on modifier release only

If a user lets go of the modifier (e.g. alt) before the 'normal' key (e.g.
tab) when window-cycling, we do not end the cycling until both keys have
been released.  If we end the window-cycling on release of the modifier
only, some XWayland clients such as hexchat realise that tab is pressed
(even though we did not forward the event) and because we absorb the
equivalent release event it gets stuck on repeat.

Just to clarify the position here: Issue #176 describes a behaviour
whereby dmenu gets stuck on repeat after being launched with a keybind.
This patch does not resolve that issue but reflects that in Wayland, the
client is responsible for implementing "key repeat".

Changing the key repeat rate/delay in (labwc/labwc@e62bb51) was dirty fix
that need should never have been made.

output: Call do_output_layout_change() at end of new_output_notify().

This fixes an issue with the wlr_output_cursor not being properly
initialized on new outputs, because wlr_output_layout.events.change
is triggered too soon, before the wlr_output_cursor exists.

src/menu.c: Clamp separator width

Reported-by: @Flrian

src/keyboard.c: reflow comment to shorten line

src/keyboard.c: stored handled keys as bound when window-cycling

...and changing TTY

Revert "keyboard: cancel repeat when handling key-bind"

This reverts commit e62bb51bfb33ee520e800cf98553d766824fe9cf.

Fixes #510

Merge pull request #552 from jlindgren90/xmalloc

common: Add more friendly memory utilities

common: Expand comment for znew/znew_n()

common: Add znew/znew_n() macros

keyboard: Fix SIGSEGV that showed up in out-of-memory fuzzing

Stack trace:

    #0  xkb_keymap_ref (keymap=keymap@entry=0x0) at ../libxkbcommon/src/keymap.c:61
    #1  0x00007f53a344ab99 in wlr_keyboard_set_keymap (kb=kb@entry=0x5571af8cb9a0, keymap=keymap@entry=0x0)
        at ../types/wlr_keyboard.c:174
    #2  0x00005571ade057e0 in keyboard_init (seat=0x7ffca0389680) at ../src/keyboard.c:229
    #3  seat_init (server=0x7ffca0389570) at ../src/seat.c:307
    #4  server_init (server=0x7ffca0389570) at ../src/server.c:308

action: Fix warning about freeing a (const char *)

common: Add additional memory utilities (xzalloc() etc.)

action: Allow explicit None action without warning

cursor: Use enum for server set cursor names

This mainly prevents having to use strcmp() on every mouse move.

cursor: Move cursor specific function definitions into cursor.h

seat: Fix use-after-free in touch_finish()

keyboard: Fix use-after-free in keyboard_finish()

action: Select resize edges for Resize triggered by keybind

cursor: Split out cursor_button_press() and cursor_button_release()

config: s/OSD/OnScreenDisplay/ in <theme><font place="OSD">

...to comply with Openbox 3.6 spec [1]

"OSD" is still honoured to maintain backward compatibility.

[1]: http://openbox.org/wiki/Help:Configuration#Theme

config: Add support for font slant and weight

cursor: Store view + resize edges for move/resize in press event

v2: Restore previous behavior for keybinds

cursor: Add struct cursor_context and clean up code

v2:
 - Add surface to cursor_context, rename c -> ctx
 - Factor out determine_resize_edges()

Merge pull request #533 from jlindgren90/fix-gtk-menus

cursor: Fix GTK3 menus when keeping the button pressed

cursor: Allow leave/enter events within the same XDG toplevel

Attempting to open a GTK3 menu and activate a menu item in it,
using a single mouse motion (press-move-release), was broken due
to GTK apparently expecting to receive leave/enter events when the
cursor enters the menu (XDG popup).

To fix the issue, allow leave/enter events when the cursor is
moved between an XDG toplevel and popups of the same.

v2:
 - Use (struct view *) as proxy for toplevel in comparisons
 - Update seat->pressed.surface when entering/leaving popups
v3:
 - Go back to using get_toplevel() rather than (struct view *)

xdg-popup: Check for NULL from wlr_xdg_surface_from_wlr_surface()

Also eliminate struct view_child and replace it with a simple
(struct view *)parent_view field.

seat: Listen for destroy signal of pressed.surface

Merge pull request #540 from jlindgren90/cursor-fixes

cursor: Factor out cursor_update_common() and fix some glitches

cursor: Factor out cursor_update_common() and fix some glitches

Fix a couple of glitches seen when exiting interactive move/resize:

 - Cursor briefly set to left_ptr rather than the correct cursor image
 - Cursor not updated if the view being moved/resized is destroyed

Also make sure to exit interactive mode if the view is going fullscreen
(labwc gets very confused otherwise).

Code changes in detail:

 - Factor out set_server_cursor() which will set the correct cursor
   image for non-client areas (either XCURSOR_DEFAULT or one of the
   resize cursors).
 - Unify the logic from cursor_rebase() and process_cursor_motion by
   factoring out cursor_update_common().  This corrects some logic
   discrepancies between the two, which should be a good thing(TM).
 - Remove the extra cursor_set(XCURSOR_DEFAULT) from interactive_end()
   and instead rely on cursor_update_focus() to do the right thing.
 - Simplify cursor_button() by just calling interactive_end() when we
   want to exit interactive mode.
 - Call cursor_update_focus() from view_destroy() if the view had mouse
   focus or was being interactively moved/resized.

v2: Eliminate force_reenter parameters and figure out automatically
    when we need to re-enter the surface.
v3: Rename wlseat -> wlr_seat.
v4: Simplify client/server cursor logic.

rcxml.c: enable tap be default on non-touch devices

Some trackpads do not honour tap if we only enable it on touch-devices.

cursor: Handle missing cursor theme

Temporary fix for #246

This should be reverted once wlroots MR !3651 is merged.

cursor: Don't load cursor theme in server_init()

We are already doing that in seat_init() -> cursor_init()

cursor: Prevent setting the same cursor image twice

Possibly fixes #512

Reported-by: @Flrian

cursor: Prevent resetting cursor icon during Move or Resize

Reported-by: @Flrian

cursor: Allow re-enter for cursor_update_focus()

This allows forcing an application to re-set their desired cursor image.

Merge pull request #531 from jlindgren90/titlebar-fixes

Titlebar fixes

ssd: Make ssd_get_part_type() work for corner buttons

Corner buttons (WINDOW_MENU and CLOSE) are one more level down in
the scene-tree (see add_scene_button_corner() in ssd_part.c).

This fixes a minor issue where, when right-clicking on the CLOSE
button, the client-menu would be displayed in the wrong location.

config: Redefine Title context to include blank areas of Titlebar

... and use Title for the Drag (Move) and DoubleClick (Maximize)
titlebar actions, which are unexpected when the cursor is over one
of the window buttons.

Merge pull request #523 from jlindgren90/unmanaged-cursor-focus

cursor: Update focus when unmanaged surfaces are mapped/unmapped