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

Merge pull request #526 from Consolatis/issue/317_gimp_remap

xwayland: Keep view->xwayland_surface and view->surface in sync

interactive: Don't write to view->x/y/w/h directly

Otherwise, the scene-graph isn't updated when calling
view->impl->configure(), and the view ends up in a weird
half-maximized state.

cursor: Update cursor focus on scroll events

cursor: Make cursor_rebase() private

Also allow to re-use node, surface, sx and sy.

cursor: Allow cursor button events to trigger cursor_rebase()

src/xdg.c: Only call view_moved() when required

desktop: Cycle first to topmost view if not already focused

The topmost view may not always be the focused view; for example,
when Audacious's main window is focused but the floating Search
Tool window remains on top of it.  In that case the floating window
(not the main window) should be the first view selected in the
window switcher.

cursor: Update SSD hover state in cursor_rebase()

Fixes an issue where the maximize button would remain highlighted
after maximized a window.

cursor: Update focus at various additional points

- When XDG and XWayland views are positioned (view_moved()).
- When unmanaged XWayland surfaces are mapped, unmapped, or moved.

Do not update cursor focus during an out-of-surface drag.

Along with the existing call in desktop_move_to_front(), this
should hopefully cover the majority of cases where the cursor focus
could get out-of-date, with the possible exception of layer-shell
surfaces.

src/xwayland.c: Keep view->surface in sync

src/xwayland.c: Update xsurface on client map request

xwayland: Don't center views with explicitly specified position

view: Call view_moved() immediately after map

At least for XWayland surfaces, handle_commit() is not always
called after map(), and as a result, the scene-graph node is never
positioned.

Not sure 100% if the same can occur with XDG surfaces, but the
extra view_moved() call should be harmless, so add it there too
for consistency.

xwayland: Factor out focus_next_surface() from unmanaged_handle_unmap()

- Eliminate multiple "return" paths in unmanaged_handle_unmap(), which
  were a bug waiting to happen.
- Use wl_list_for_each_reverse() rather than wl_list_for_each() to find
  the topmost (most-recently-created) unmanaged surface.
- Only call desktop_focus_topmost_mapped_view() if the unmapped surface
  was actually focused.

CI: Split jobs

CI: Additionally build without xwayland

cursor: Fix out-of-surface movement for unmanaged surfaces

xwayland: Keep unmanaged surfaces stacked on top

src/xwayland-unmanaged.c: Give keyboard focus back to topmost mapped view

Previously, when unmapping and not finding another unmanaged surface
to give keyboard focus to we were not doing anything. With this patch
we will give focus to the topmost mapped view which is the same
behavior that also happens when an usual X11 surface unmaps.

This fixes dmenu when being exited with ESC.

Reported-by: @Flrian

src/config/rcxml.c: Add support for <devault /> mousebinds

This loads default mousebinds and provides a way to keep config files
simpler whilst allowing user specific binds.

Note that if no rc.xml is found, or if no <mouse><mousebind> entries
exist, the same default mousebinds will be loaded even if the <devault />
element is not provided.

Example usage (with a slight spelling error):

  <mouse>
    <default />
    <context name="Root">
      <mousebind button="Right" action="Press">
        <action name="ShowMenu" menu="desktop-menu" />
      </mousebind>
    </context>
  </mouse>

Co-Authored-By: @johanmalm
Fixes #416

src/config/rcxml.c: Replace earlier mousebinds by later ones

src/config/mousebind.c: Fix mousebind ordering

src/config/rcxml.c: Merge default mousebinds

src/view.c: Fix minimizing the last window

Before this patch, minimizing the last un-minimized window would
not cause it to be set to inactive. This in turn would confuse panels
which depend on the 'active' flag to decide when to either activate or
minimize the clicked on window.

Reported-by: @Flrian
Predicted-by: @johanmalm

Merge pull request #307 from Consolatis/feature/tab_preview

Feature/tab preview: restore functionality after the move to scene-graph

Move OSD handler for view destruction to osd.c

src/osd.c: Integrate preview outline into OSD state

alt-tab preview: restore functionality after move to scene-graph

Prepare OSD for reimplementation of the preview feature

Chase wlroots: Culling

To update the wlroots subproject use
meson subprojects update wlroots

Chases wlroots fa7d2cb8d60ed48c44c707106c03682056ddfaca
wlr_scene: Only consider visible parts of the node when culling background

Fixes #501

osd: position preview border within SSD max extents

This makes the border visible when alt-tabbing maximized windows and
prevents it from rendering on other outputs or on top of exclusive
layer-surfaces such as panels.

Closes issue #496

view/ssd: Refactor and fix visual indication for active windows

Partially fixes #494.

Co-Authored-by: Consolatis

src/{common/font,theme}.c: Switch to cairo helpers

Disconnect view from destroying output

Fixes #497

Reported-by @Flrian

Add config option to disable preview outlines

keyboard: Hide Alt-Tab switcher when canceling via Escape

osd: Do not show window switcher on disabled outputs

src/osd.c: Add preview outlines

common/graphic-helpers: Add cairo helpers

common/graphic-helpers: Add multi_rect

labwc-theme(5): add menu.xbm

view: view_move() s/double/int/ for x and y

func output_destroy_notify add free

src/cursor.c: refactor nested if-statements

cursor: do not clamp motion coordinates for xdg-shell surfaces

...when pointer moves off the xdg-shell surface which is possible when
for example grabbing a scroll-bar or selecting text.

Clamping results in undetected pointer motion when applied in the
direction of movement (for example clamping at view->y masks vertical
movement when the y-coordinate is greater than view->y), which in turn
can appear as sporadic to the end-user.

src/cursor.c: Keep sending motion events for out-of-layer-surface

Previously, motion events for a cursor which had been pressed on a surface
and then moved out of the surface were only sent for XDG and X11 surfaces.

This patch includes layer surfaces as well.

Fixes #483

cursor: Do not clamp motion coordinates for XWayland surfaces.

X11 apps expect to be able to receive motion events outside
the window area (this is necessary for client-side move/resize
handles to work properly).  So do not clamp the motion
coordinates for XWayland surfaces.

Before this change, attempting to enlarge an XWayland window
using a client-side resize handle resulted in the window size
lagging behind the mouse cursor quite severely, since each
motion event was in effect allowed to expand the window by
only a few pixels.  The closer the initial button-press was
to the edge of the window, the worse the lag would be.

keybind: remove default alt-escape for Exit

...because too many have exited the compositor by mistake trying to get
out of alt-tab cycling or similar.

src/ssd/ssd_titlebar.c: Completely hide title when running out of space

Before this commit the window title could overflow in certain conditions,
usually when rapidly resizing. This commit fixes it by completely hiding
the corresponding scene node when there is no space available.

Reported-by: <peperino> via IRC

scaled_font_buffer.c: simplify calls to zfree()

There is no need to check foo != NULL before zfree(foo)

menu: render submenu arrows

font: remove unused font_buffer_update()

menu: do not require label attribute for highest level menu definition

Allow highest level menu definitions - typically used for root-menu and
client-menu - to be defined like this:

    <openbox_menu>
      <menu id="">
      </menu>
    </openbox>

Previously this required a label attribute (which was not used for
anything and could be an empty string) as show below:

    <openbox_menu>
      <menu id="" label="">
      </menu>
    </openbox>

Closes issue #472

src/xwayland-unmanaged.c: Prevent racing configure event to reposition

Fixes #469

Reported-by: @Flrian
Suggested-by: @jlindgren90

README.md: fix broken link for 'environment' man page

- re: https://github.com/labwc/labwc.github.io/pull/1

CI: auto-update freebsd-vm within major version

Reduce the churn by transparently picking up bustage fixes.

src/config/session.c: Handle allocation failures

Reported-by: @heroin-moose

src/config/session.c: Update dbus / systemd activation environment

This allows xdg-desktop-portal-wlr to work out of the box for screen-recording.
If systemd or dbus is not available the environment update will fail gracefully.

This patch will set XDG_CURRENT_DESKTOP=wlroots but a user may change this by
either having the environment variable set before starting labwc or by having
a different value set in ~/.config/labwc/environment.

Based on PR #461 by @Joshua-Ashton

src/view.c: Fix output handling with no outputs available

Reported-by: @Flrian

xwayland: Process unmanaged client configure requests more quickly

This is a similar fix to 065c37d3f5ee but for unmanaged windows.  The issue
could be seen for example when moving the undocked Search Tool window in
Audacious.  For unmanaged windows, we don't track any pending move/resizes,
so just process all client configure requests immediately.

Merge pull request #457 from Consolatis/fix/xwayland_seat

xwayland: (Re)set seat when xwayland is ready

CI: Use MacOS-12 for FreeBSD runner

xwayland: (Re)set seat when xwayland is ready

For some reason wlroots will reset the seat assigned to xwayland
to NULL whenever Xwayland terminates. This patch restores the seat
whenever Xwayland is ready again.

Fixes #166
Fixes #444

Thanks @droc12345 for figuring out the actual issue.

CI: Update FreeBSD runner

xwayland: Process move-only client configure requests more quickly

For a move-only client configure request, treat it similarly to
view_move() by updating the scene-graph immediately, rather than waiting
for handle_commit().  Move-and-resize requests are handled the same as
before.

This (mostly?) fixes the glitchiness that was noticeable when dragging
an undecorated XWayland window (e.g. Audacious in Winamp mode).

Also:

- Reduce some code duplication in handle_request_configure() by
  simply calling configure(), as suggested by @johanmalm in #428.
- Factor out common logic after a move and/or resize into view_moved().

resistance: Do not consider edges of disabled outputs

cursor: Move resistance_resize_apply() before view_adjust_size()

This ensures that screen edge resistance does not result in sizes
that violate WM_SIZE_HINTS constraints.

Also correct a coding style issue nearby.

Merge pull request #454 from Joshua-Ashton/bump-fd-limit

main: Increase FD limit

main: Increase FD limit

This defaults to 1024, which is tiny, but is a requirement
for processes using the deprecated `select` function.

We must reset this back whenever we fork to start a new process,
as this is inherited, and breaks applications using `select` otherwise.

Signed-off-by: Joshua Ashton <joshua@froggi.es>

common: Add helpers for raising/lowering FD limit

Signed-off-by: Joshua Ashton <joshua@froggi.es>

buf.c: fix bug in environment variable expansion

Allow underscore in environment variable names.

Closes issue #439

Helped-by: @Consolatis
...who both found the bug and told us how to fix it :)

NEWS.md: add notes on 0.5.3

NEWS.md: add notes on 0.6.0

cursor: Fix crash when creating a cursor constraint

Fix crash when creating a cursor constraint and there is no currently
focused view.

Fixes a crash starting Warhammer 40,000: Dawn of War II (15620)

config/rcxml.c: Fix parsing of adaptiveSync setting

CI: upgrade FreeBSD to 13.1

FreeBSD doesn't support /latest and /quarterly package repos on EOL
versions. 13.0 reaches EOL on 2022-08-31, so avoid CI breakage.

CI: Add missing deps for Void Linux runner

src/view.c: Prevent crash by killing a moving application

When a view is destroyed labwc calls interactive_end(view) which
may reposition the view which is partly destroyed and doesn't own
any surface anymore. To prevent this scenario from happening don't
call interactive_end() at all and just reset server->grabbed_view
and server->input_mode directly.

Before this patch, the bug could be reproduced by:
- xcalc &
- sleep 5; killall xcalc
- move the xcalc window completely to one of the edges

The change in src/xwayland.c is not required for this bug
to be fixed but may prevent something similar in the future.

README.md: refresh and refactor

- Use &nsbp; in links with spaces
- Change headings from verbs to nouns
- Combine build and install headings
- In 'What is this', use the text on the labwc website
- Remove video links; one wasn't working and the other were very old
- Use more recent screenshot
- Simplify the 'labwc supports the following' bullet points
- Split dependencies into build and run-time
- Remove dependecy version numbers and let meson.build be the source of
  truth
- Simplify config file table by taking out middle column
- Split the configuration section into configuration and theming
- s/roadmap/scope
- Change man page links from docs/ to labwc.github.io/

src/output.c: Always react to new output configuration

Without this fix we will never react to output configuration changes
via wlr-randr and friends.

If output_config_apply() commits the new config, handle_output_layout_change()
is called but doesn't do anything because the config is still pending.

This patch moves the code into its own function do_output_layout_change()
which additionally gets called after all output configs are committed.

The original handler is turned into a wrapper around do_output_layout_change().

Reported-by @heroin-moose

xwayland.c: fix position bug

Set node position in the configure/set_geometry handlers when moving a
window in response to a client request.

Steps to reproduce weird positioning fixed by this patch:

1. Start leafpad
2. Open Help->About
3. Move the dialog
4. Close the dialog
5. Open it again
6. Drag it and observe a jump in position

There is also an xwayland PyQt5 script in PR #428 which demonstrates
jumpy position.

src/action.c: Convert action->arg to a list of action_arg

In preperation for Actions that require multiple arguments

src/view.c: Fall back to default geometry when changing state

This makes sure that applications starting in maximized of fullscreen
mode always have their natural_geometry set to sensible values.

Partly fixes #403

(Partly) fix handling of client-initiated configure requests

- Add missing call to wlr_scene_node_set_position() in
  unmanaged_handle_commit() -- this fixes moving unmanaged XWayland
  windows.

- Update view->pending_move_resize when we receive a configure request
  for a managed XWayland surface -- this fixes moving managed but
  undecorated XWayland windows.

- Also update view->pending_move_resize when we move a surface while a
  configure request is already pending -- this fixes a discrepancy
  between displayed and actual position for XWayland windows that try to
  set their own initial position, but then get overridden by labwc's
  positioning.

Moving undecorated XWayland windows is still really glitchy -- it appears
that an XWayland window gets sent incorrect mouse motion coordinates when
there is a pending configure request moving the window itself.

Restore original geometry when moving a tiled window

Fixes #391

Rename unmaximized_geometry to natural_geometry

src/view.c: Unmaximize on SnapToEdge

src/view.c: Prevent SnapToEdge while in fullscreen

src/view.c: Re-arrange tiled windows on un-fullscreen and un-maximize