gdk/wayland: Drop surface checks on wl_pointer.leave handling

Even though the argument is non-nullable, GTK sometimes incurs in that
by itself by destroying the surface while the event is in flight. This
is the case of popping down a GtkDropdown. When this happens we simply
ignore the crossing event, but we should let it through instead, the
compositor did not send it in vain and we possibly still have pointer
state to undo.

Drop the surface checks, so that the event is propagated along GTK.

gtkeventcontroller: Fix internal enum documentation

Copy and paste typo.

gtkgesturerotate: Filter touchpad holds events

Since the pinch touchpad gestures already have begin/update/end phases,
this gesture has no actual use for these events.

gtkgesturerotate: Filter touchpad holds events

Since the pinch touchpad gestures already have begin/update/end phases,
this gesture has no actual use for these events.

gdkevent: Give touchpad hold events a sequence

Following what was done for pinch/swipe events, give hold gestures their
own distinct sequence as well. Without this it was NULL, which was already
distinct to other touchpad gestures.

gtkgesture: Do not coalesce hold events with other touchpad gestures

This delaying of the cancel event was made to avoid intermediate cancellation
for >=2fg hold gestures followed by pinch/swipe gestures, and it worked as
long as everything was considered to have the same sequence.

Since each pinch/swipe pointer gesture now gets its own sequence, this no
longer applies, nor works. This results in zoom/rotate/swipe gestures being
stuck since the sequence for the touchpad events changes mid-gesture.

Sticking to this pattern of giving touchpad gestures their own sequence,
these hold events cannot be assumed to coalesce with other touchpad gestures,
it is better to let it propagate altogether so that both the hold gesture
and the incoming gesture trigger coherent begin and end/cancel phases.

In the worst case, this results in "::begin, ::cancel, ::begin , ..." before
triggering a touchpad gesture, but the extra begin/cancel ought to be a safe
no-op in widgets.

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5003

Merge branch 'ebassi/textbuffer-private-api' into 'main'

Move private function out of the AT-SPI a11y backend

See merge request GNOME/gtk!4924

Re-enable the TextBuffer wrap-mode serialization test

The private serialization function is now available on all platforms.

Move private function out of the AT-SPI a11y backend

The textbuffer test is calling into a function defined by the AT-SPI
accessibility backend. As of commit 4ddf1b70 we only build and run the
test on Linux, but the function in question isn't really
accessibility-related: it's just a serialization function.

Update Ukrainian translation

Merge branch 'cleanup-ffmpeg' into 'main'

ffmpeg: add audio playback through avdevice

See merge request GNOME/gtk!4852

ffmpeg: add audio playback through avdevice

Merge branch 'mount_operation' into 'main'

mountoperation: Fix authentication dialog

Closes #5059 and #5058

See merge request GNOME/gtk!4899

Merge branch 'win32-nahimic-nv-notes' into 'main'

running.md: Mention about issue with Nahimic on Windows

Closes #4113

See merge request GNOME/gtk!4915

running.md: Mention about issue with Nahimic on Windows

Let people know that they will need to use GTK with the Nahimic service
disabled or OpenGL disabled or put their GTK application into the Nahimic
backlist, or try to use GLES, since there is a known issue in the Windows
nVidia graphics drivers and Nahimic that causes GL operations to fail,
causing crashes in operations such as window resizes.

This will close issue #4113--sadly, there is nothing we can do within
GTK to fix the issue.

Merge branch 'label-tabs' into 'main'

Label tabs

See merge request GNOME/gtk!4914

Merge branch 'otte-main-patch-86462' into 'main'

Fix g_object_new(GTK_TYPE_SNAPSHOT)

See merge request GNOME/gtk!4919

snapshot: Don't critical on dispose

When there's remaining state that draws nothing, it returns no
rendernode. So do a NULL-check before unreffing it.

Fix g_object_new(GTK_TYPE_SNAPSHOT)

GJS apparent likes g_object_new() over actual constructors, so make that work.

Add GtkLabel:tabs

This adds more powerful formatting capabilities to GtkLabel
without costing us much, since Pango already supports tabs.

Merge branch 'wip/chergert/builder-expose-object-fix' into 'main'

builder: gracefully handle re-exposing builder object

See merge request GNOME/gtk!4913

gtkbuilder: update gtk_builder_expose_object() docs

gtkbuilder: update gtk_builder_extend_with_templat() docs

gtkbuilder: check for existing object before extending template

If gtk_builder_expose_object() is called twice with the same name, it will
result in a g_critical(). This improves that situation by checking for the
object before exposing additional times.

This turns out to be handy in situations where templates are expanded
multiple times, such as application-side implementations of UI merging.

Merge branch 'fix-gdk-win32-dnd' into 'main'

Fix DnD on Windows

Closes #4498

See merge request GNOME/gtk!4900

Update German translation

Update Abkhazian translation

Merge branch 'alatiera/node-flatpak' into 'main'

flatpak: add a NodeEditor demo

See merge request GNOME/gtk!4898

Merge branch 'ebassi/file-list-fixes' into 'main'

Add constructors for GdkFileList

See merge request GNOME/gtk!4907

Add constructors for GdkFileList

C API users can keep dealing with the implicit equivalence of
GdkFileList and GSList, but language bindings have no idea that one type
is another, and none of them exposes GSList as a type anyway, so they
will need a way to construct a GdkFileList.

Instead of making GdkFileList mutable, and re-implement GSList, we only
provide a constructors pair that lets you create a GdkFileList from a
linked list or from an array.

Merge branch 'barthalion/gnome-runtime-images-quay' into 'main'

Update (or drop) the Docker image used by Flatpak job

See merge request GNOME/gtk!4906

Update (or drop) the Docker image used by Flatpak job

The gnome-runtime-images have been recently migrated to Quay. This is already reflected in the template.

Please note this MR has been created semi-automatically. If it doesn't make sense, feel free to close it.

Updated Spanish translation

Merge branch 'wip/chergert/disable-sysprof-agent' into 'main'

build: configure subproject sysprof with -Dagent=false

See merge request GNOME/gtk!4896

build: configure subproject sysprof with -Dagent=false

Sysprof has a new -Dagent=true build option which allows installing a
/usr/bin/sysprof-agent program (simimlar to sysprof-cli). It provides a
P2P D-Bus API to the process which can control subprocesses. It's used by
IDE tooling to have more control across container boundaries.

However, we do not need it for GTK CI.

Merge branch 'markand-main-patch-04776' into 'main'

gtk: Missing end of code block

See merge request GNOME/gtk!4902

gtk: Missing end of code block

Add Abkhazian translation

Merge branch 'gridview' into 'main'

Gridview fix rubberbanding issues

Closes #3462 and #3445

See merge request GNOME/gtk!4688

listview: Fix selection for last item in get_items_in_rect

Make the last item dependent on y + height

listview: Allow starting rubberband in empty space below last row

Rubberband does not work when initiated past the last row
(warning is printed "Could not start rubberbanding: No item).

Clamp y at the max height of the widgets in the listview

listview: Return an empty bitset when selecting only empty space

Return an empty bitset if the user selects exclusively below the last
row.

gridview: Fix typo in docstring for gtk_grid_view_get_cell_at_y()

listbase: Clear selection if rubberband selected an empty set

If rubberband returns an empty bitset the selection should be cleared
unless the shift or ctrl key is held

gridview: Return an empty bitset when selecting only empty space

Return an empty bitset if the user selects exclusively below the last
row.  No need to calculate selection.

gridview: Fix get_items_in_rect() selection

last_row should be dependent on y + height, not y

gridview: Allow starting rubberband in empty space below last row

Rubberband does not work when initiated past the last row
(warning is printed "Could not start rubberbanding: No item).

Clamp y at the max height of the widgets in the gridview

Fixes: #3462

gridview: Limit rectangle to gridview columns

The function gtk_grid_view_get_items_in_rect() erroneously calculates
columns less than 0 and greater than n_columns when the user attempts
to rubberband all the way to the left or right respectively.  This
causes the rubberband to persistent and creates unexpected behavior.
Limit the rows to a minimum of 0 and maximum of n_columns - 1.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/3445

gridview: Move gtk_grid_view_computer_total_height() up for reuse

Merge branch 'widget-docs-fix' into 'main'

gtkwidget: Fix a small typo in a docs comment

See merge request GNOME/gtk!4901

gtkwidget: Fix a small typo in a docs comment

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>

Update Georgian translation

Fix DnD on Windows

DnD under Windows needed 3 fixes to work with Gtk.DropTarget.

1. The droptarget_w32format_contentformat_map list never gets
filled so the gdk_win32_drop_read_async throws
"No compatible transfer format found".
This is an easy fix and done the same way in the win32 clipboard code.

2. After a drop no gdk_drop_emit_leave_event gets emitted.
This causes a second drop to trigger a bunch of assertion
'self->drop == drop' failed because the first drop is still active.
This is also an easy fix and done the same way by the macos backend.

3. Handling gdk_drop_status/gdk_drop_get_actions interaction.
In gtk_drop_target_do_drop the code
```gdk_drop_finish (self->drop, gdk_drop_get_actions (self->drop));```
calls the finish operation with the actions of the drop which triggers
```g_return_if_fail (gdk_drag_action_is_unique (action));```
in gdk_drop_finish. The code assumes that GdkDrop::actions gets
narrowed down by calling gdk_drop_status. This is hard to assure
because at the same time gdk_drop_get_actions is used by
gtk_drop_target_accept to figure out if a drag is accepted.
GdkDrop::actions serves a double purpose here as the supported source
actions and the currently agreed on action. Both the x11 and the
wayland backend get this wrong somewhat too. Under wayland/x11 when
a drag coming from a source that supports both MOVE and COPY is
first hovering a drop target that only supports COPY it is afterwards
no longer accepted by other drop targets only accepting MOVE.
Under x11 this is permanent for this drag but with wayland the drag
recovers when hovering other widgets. The win32 backend now sets the
supported source actions before any enter/move/drop and narrows them
down in gdk_win32_drop_status.

The patch only touches the win32 backend and fixes all three issues,
for me restoring DnD under windows.

Closes #4498

mountoperation: Free user_widgets list upon dialog destruction

There's a list user_widgets that contains all of the entries and
selections during authentication.  This is only freed  upon
finalizing the GtkMountOperation.  It's possible (and true for the
GVFS SMB implementation) that a MountOperation can have the
gtk_mount_operation_ask_password_do_gtk () function called multiple
times (i.e. bad password).  The user_widgets list grows with now
invalid pointers to old widgets (causing unexpected behavior and
seg faults).

Free the user_widgets list upon dialog destruction, we don't need it
anymore.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5059
Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5058

flatpak-build.sh: change the buildtype to debugoptimized

So that the flatpaks will include debuginfo and make debugging
easier.

flatpak: add a NodeEditor demo

Merge branch 'content-fit-since' into 'main'

picture: Fix Since and Deprecated annotations for content-fit

See merge request GNOME/gtk!4897

picture: Fix Since and Deprecated annotations

gi-docgen will parse four or more whitespaces of indentation as code blocks,
two whitespaces were used.

This was missing in https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/4873.

mountoperation: Select "Anonymous" check button by default

It is already explicitly assumed that anonymous authentication will
be used when available, but it is not clear to the user since neither
of the check buttons are selected.  Select the Anonymous check button
by default.

Update Catalan translation

Merge branch 'fix-controller-key-event-propagation-file-chooser-widget' into 'main'

GtkFileChooserWidget: Propagate captured key events when the location entry is focused

Closes #4880

See merge request GNOME/gtk!4674

Merge branch 'drag-action' into 'main'

tree/iconview: Use a unique drag action in drag_enter/motion callbacks

Closes #3187

See merge request GNOME/gtk!4882

Merge branch 'treeview-drag-ancestor' into 'main'

treeview: Check for column header area in row click/drag gestures

Closes #4453

See merge request GNOME/gtk!4885

Merge branch 'matthiasc/for-main' into 'main'

css: Fix handling of transform values

See merge request GNOME/gtk!4890

Merge branch 'wip/carlosg/use-press-serial' into 'main'

gdk/wayland: Use last implicit grab serial for DnD start requests

Closes #5048

See merge request GNOME/gtk!4889

gdk/wayland: Stop storing the last serial in GdkWaylandDisplay

This is not used anymore, so we can drop the serial being stored
here.

Merge branch 'drop-target-unique' into 'main'

droptarget: Provide a unique action for gdk_drop_finish

Closes #5051

See merge request GNOME/gtk!4888

Merge branch 'tree-popover-scroll' into 'main'

treepopover: Add support for scrolling

Closes #3674 and #4877

See merge request GNOME/gtk!4887

css: Improve test coverage

Among other things, this includes a test for the fix
in the previous commit.

css: Fix handling of transform values

When computing a transform value, there is nothing
to do, but we still need to copy the matrix from
src to dest, since it depends on the other transforms
in the array whether we are using the src or the
dest in the end.

This fixes cases like
-gtk-icon-transform: perspective(100px) matrix(1,2,...);
which would otherwise end up with a zero matrix.

gtk: Improve test coverage

sortlistmodel: Add more tests

filterlistmodel: Add more tests

gdk/wayland: Use last press serial for clipboard/primary selections

This serial observes key and button presses, we can use that instead
of relying on the last serial which might come from other
events/devices.

gdk/wayland: Use last implicit grab serial for DnD start requests

This serial should be that from a button press/touch down/etc, use
the last implicit grab here, which will presumably be from the same
device that triggered the event.

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5048

treepopover: Add support for scrolling

Add support for scrolling large tree popovers in a similar fashion to GtkPopoverMenu.

Fixes #3674
Fixes #4877

droptarget: Provide a unique action for gdk_drop_finish

gdk_drop_finish() requires a unique drop action. Reuse the existing
make_action_unique() function to get one.

Fixes #5051

Update Occitan translation

Merge branch 'picture-suggest-content-fit' into 'main'

picture: Remove keep-aspect-ratio mention in the doc

See merge request GNOME/gtk!4886

treeview: Check for column header area in row click/drag gestures

Bail out if a row click/drag gesture is performed in the column header area,
since separate gestures handle column headers.

Fixes #4453

picture: Remove keep-aspect-ratio mention in the doc

We were still suggesting to use it in the main doc page. Suggest using
content-fit instead.

Merge branch 'matthiasc/for-main' into 'main'

filterlistmodel: Add a test

See merge request GNOME/gtk!4884

slicelistmodel: Improve a test

Make sure that replacements in the underlying model
don't affect the slice list model if they are outside
the slice window.

sortlistmodel: Add a test

Add  a test to prove that additions and removals
in the underlying model show up as expected.

filterlistmodel: Add a test

Test that items added in the underlying model show
up in the filter model (if they pass the filter).

Test removals as well.

See #5050

tree/iconview: Use a unique drag action in drag_enter/motion callbacks

Functions already exist for providing a unique drag action for gdk_drop_finish().
Reuse these functions in the drag_enter/motion callbacks, since they require
a unique action as the return value.

Fixes #3187

Merge branch 'matthiasc/for-main' into 'main'

widget-factory: Make the icon work uninstalled

See merge request GNOME/gtk!4881

widget-factory: Make the icon work uninstalled

Include the app logo as a resource, so we can show
it even when uninstalled.

Merge branch 'fix-wrong-label-wrap-in-popover' into 'main'

popover: Correctly substract arrow height when measuring popover widget

See merge request GNOME/gtk!4880

Merge branch 'dnd' into 'main'

DnD Fix Preferred Action for X11

Closes #4259

See merge request GNOME/gtk!4785

gdkdrop-x11: Use the preferred action if possible

The XDND suggested action is a relic from when the source would control
the action for a drop.  With the new GtkDropTarget the target decides
the action (not the source).  That means the all of the returned
results from the ::enter and ::motion handlers will be unexpectely
ignored. Prefer to use the preferred action over the x11 suggested action.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/4259

gdkdrop-x11: Fix preferred action

gdk_x11_drop_update_actions() sets actions to
drop_x11->suggested_action when !drop_x11->xdnd_have_actions
and then sets it again to drop_x11->suggested_action if it is.
If xdnd_have_actions is true use xdnd_actions.

popover: Correctly substract arrow height when measuring popover widget

For default popover arrow position and default height-for-width layout mode,
natural_width is calculated first with for_size=-1 and orientation=HORIZONTAL,
at the end of gtk_popover_measure() natural_width  won't be added with tail_height.

Then to measure with for_size=natural_width and orientation=VERTICAL, obviously
for_size shouldn't be substract with tail_height.

The wrong logic will force content in popover gets less width and then text labels
in popover may get wrapped unnecessarily.

Merge branch 'content-fit-reftests' into 'main'

reftests: Add content-fit picture tests

See merge request GNOME/gtk!4879

reftests: Add content-fit picture tests

Merge branch 'test-coverage' into 'main'

gtk-demo: Fix icon theme breakage

See merge request GNOME/gtk!4878

gtk-demo: Make the icon work uninstalled

Include the app logo as a resource, so we can show
it even when uninstalled.

gtk-demo: Add a winning sound to the puzzle

We should celebrate every win.

gtk-demo: Fix icon theme breakage

The starred emblem silently went away :(

gdk: Improve test coverage