Add a test for treelistmodel row collapse

Test that we can expand and collapse a row, and then
add another child below it, without crashing.

Adapted from the testcase in #4595.

This tests the fix in the previous commit.

treelistmodel: Fix handling of collapsed nodes

When we collapse a node, we clear out the children,
but we were not disconnecting the signal handler on
the child listmodel, leading to bad outcomes when
that model is persistent and changing.

Fixes: #4595

Merge branch 'tweak-option-docs' into 'main'

Tweak the vulkan option description

See merge request GNOME/gtk!4976

Tweak the vulkan option description

The thing that is experimental is our renderer,
not the vulkan API.

Update Icelandic translation

(cherry picked from commit a27ae5362c81e4395e962a9e0636a9f92c50c6ef)

Update Icelandic translation

(cherry picked from commit e6d3c58ccfeba0840b4849238f431124a1903605)

Merge branch 'win32-gir' into 'main'

Generate gtk/GdkWin32-4.0.gir

Closes #5112

See merge request GNOME/gtk!4965

Updated Lithuanian translation

Merge branch 'fix-calc-value-free' into 'main'

css: Fix freeing of calc values

Closes #5130

See merge request GNOME/gtk!4974

css: Fix freeing of calc values

We were getting our memory allocations mixed
up here.

Fixes: #5130

Merge branch 'jpeg-memory-limit' into 'main'

Bump the jpeg memory limit

See merge request GNOME/gtk!4973

Bump the jpeg memory limit

Commit 59f6c50df8d4d9 set the memory limit to 100M,
which turns out to exclude some large, valid jpegs.
So, bump things to 300M, matching what was done
in gdk-pixbuf.

Merge branch 'no-emoji-insert' into 'main'

text: Respect no-emoji hint fully

See merge request GNOME/gtk!4972

text: Respect no-emoji hint fully

We were disabling the insert-emoji action when the
no-emoji input hint is set, but the Ctrl-. shortcut
was bypassing the action and kept working. Make
the shortcut activate the action instead.

Fixes: #5123

Merge branch 'wip/smcv/log-writer' into 'main'

icontheme test: Assert that the subprocess passed

See merge request GNOME/gtk!4969

icontheme test: Assert that the subprocess passed

Noticed while backporting GLogWriterFunc fixes to 3.24.x (#5119).

Signed-off-by: Simon McVittie <smcv@debian.org>

Update Serbian translation

Update Galician translation

Merge branch 'gdk-win32-fix-handling-of-input-region' into 'main'

GdkWin32: Fix handling of WM_NCHITTEST

Closes #5114

See merge request GNOME/gtk!4966

Update Georgian translation

Merge branch 'fix-emojichooser-keynav' into 'main'

emojichooser: Fix arrow keynav

Closes #5076

See merge request GNOME/gtk!4967

emojichooser: Fix arrow keynav

When some of the Emoji have been filtered out by
a search term, arrow keynav would behave oddly and
get stuck in invisible sections. Fix this by ignoring
any filtered out children when moving between
sections for arrow keynav.

Fixes: #5076

Fix typo

And reword comment

GdkWin32: Fix handling of WM_NCHITTEST

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

build-sys: generate GdkWin32.gir

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Merge branch 'mingw' into 'main'

Cross-building with mingw & fixes

See merge request GNOME/gtk!4341

build-sys: do not install gdkwin32keys.h

Fixes: commit 89ad7893adb25a79 ("gdk: Make GdkKeymap a private api")
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk/win32: remove old declaration

The function is gone since commit ea65abc7e2cd ("Rewrite
GdkWin32Keymap (load table directly from layout DLL)")

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Merge branch 'wip/otte/for-main' into 'main'

gdk: Plug a memleak

Closes #5110

See merge request GNOME/gtk!4964

gdk: Plug a memleak

Free GdkIOPipe struct when freeing it
Don't just free all its members.

Fixes #5110

Merge branch 'treeview-search-focus' into 'main'

treeview: Always call grab_focus_without_selecting on search entry

Closes #4551

See merge request GNOME/gtk!4961

Merge branch 'remove-msys2-ci-subproject-workaround' into 'main'

CI: remove workaround for some subproject builds failing on Windows

See merge request GNOME/gtk!4923

Merge branch 'wip/carlosg/im-fixes' into 'main'

gtkimcontextwayland: Refactor handling of client updates

Closes #5106 and #5105

See merge request GNOME/gtk!4962

gtkimcontextwayland: Refactor handling of client updates

Currently, the wayland IM context sends zwp_text_input_v3.commit from
a number of places, and some of them with partial data. In order to
make client state updates "atomic" and complete, make the communication
happen over an unified notify_im_change() function that happens on
a narrower set of circumstances:

  - The GtkIMContext is reset
  - The GtkIMContext is just focused
  - The gesture to invoke the OSK is triggered
  - The IM context is reacting to changes coming from the compositor

Notably, setting the cursor location or the surrounding text do not try
to commit state on their own, and now will be flushed with the corresponding
IM update or reset. But also, these requests won't be prevented from
happening individually on serial mismatch, instead it will be the whole
state commit which is held off.

With these changes in place, all client-side updates are notified
atomically to the compositor under a single .commit request.

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

gtktextview: Shuffle the places doing IM reset

During text widget manipulation (inserting or deleting text via keyboard)
the IM context is reset somewhat early, before the actual change took place.
This makes IM lag behind in terms of surrounding text and cursor position.

Shuffle these IM reset calls so that they happen after the changes, and
ensure that the IM is actually reset, since that is currently toggled on
a pretty narrow set of circumstances.

Also, fix a bug during GtkEventControllerKey::im-update where the condition
on cursor position editability to reset the IM context was inverted.

treeview: Always call grab_focus_without_selecting on search entry

The search text entry is always a GtkText widget, not GtkEntry. If a
custom search entry is set, this part of the code is never reached.

Fixes #4551

ci: add fedora-mingw64 job

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

build-sys: fix linker warning on win32

[196/296] Linking target testsuite/gtk/builder.exe
/usr/lib/gcc/x86_64-w64-mingw32/11.2.1/../../../../x86_64-w64-mingw32/bin/ld: warning: --export-dynamic is not supported for PE+ targets, did you mean --export-all-symbols?

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gtktext: Shuffle the places doing IM reset

During text widget manipulation (inserting or deleting text via keyboard)
the IM context is reset somewhat early, before the actual change took place.
This makes IM lag behind in terms of surrounding text and cursor position.

Shuffle these IM reset calls so that they happen after the changes, and
ensure that the IM is actually reset, since that is currently toggled on
a pretty narrow set of circumstances.

Merge branch 'wip/smcv/rej' into 'main'

demos: Remove detritus from a patch not applying cleanly

See merge request GNOME/gtk!4957

demos: Remove detritus from a patch not applying cleanly

I assume this was committed by mistake. It isn't used, and some
packaging systems will automatically remove it during `clean`.

Signed-off-by: Simon McVittie <smcv@debian.org>

Update Catalan translation

Merge branch 'overshoot-win32-fix' into 'main'

GdkWin32: Support high resolution scroll wheel events

See merge request GNOME/gtk!4950

Merge branch 'gtktreeview-adjustment' into 'main'

treeview: Do not set top row adjustment value while animating

Closes #4550

See merge request GNOME/gtk!4940

Merge branch 'treeview-widget-coords' into 'main'

treeview: Use widget coordinates where required

See merge request GNOME/gtk!4939

Merge branch 'wip/otte/for-main' into 'main'

listitemwidget: Actually unref the listitems

Closes #5102

See merge request GNOME/gtk!4955

listitemwidget: Actually unref the listitems

Fixes #5102

GdkWin32: Support high resolution scroll wheel events

Update Polish translation

Merge branch 'fix-gears-leak' into 'main'

gtk4-demo: Fix a memory leak

See merge request GNOME/gtk!4954

gtk4-demo: Fix a memory leak

The gears demo was leaking its vertices.

Merge branch 'wayland-cursor-texture-leak' into 'main'

wayland: Avoid leaking cursor textures

See merge request GNOME/gtk!4953

wayland: Avoid leaking cursor textures

When we fall back to the default cursor and load it
from a resource, we need to free the texture.

Merge branch 'inscription-leak-fixes' into 'main'

inscription: Plug a memory leak

See merge request GNOME/gtk!4952

inscription: Plug a memory leak

PangoFontMetrics also need to be freed.

inscription: Plug a memory leak

PangoLayoutIter needs to be freed.

Merge branch 'fix-cups-backend-leak' into 'main'

Add more valgrind suppressions

See merge request GNOME/gtk!4951

cups printbackend: Plug memory leaks

This loop has many early exits, and all of them
need to free this info object that gets allocated
at the beginning.

Add more valgrind suppressions

This gets all GQuark functions out of the logs.

Merge branch 'fix-a11y-leak' into 'main'

a11y: Fix a memory leak

See merge request GNOME/gtk!4949

a11y: Fix a memory leak

We need to free the queued context list in dispose
if we didn't get to register the contexts, and we also
need to free the list properly when we do get to
register them.

This showed up in valgrind as leaked GList structs.

Update Croatian translation

Update Croatian translation

Update Croatian translation

Update Georgian translation

Merge branch 'fix-leak' into 'main'

gtk/css: data_url_path fix leak in error path

See merge request GNOME/gtk!4947

gtk/css: data_url_path fix leak in error path

Fixes 7900032f3067166502978413ed2447c68dc83771

4.7.2

Merge branch 'improve-css-test' into 'main'

css: Propagate charset conversion errors

See merge request GNOME/gtk!4943

tests: Check the error first

The error contains useful information when
things fail, so check it first to get useful
output in case of failure.

css: Propagate charset conversion errors

Lets not throw away informtation that
helps to figure out why things fail.

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

jpeg loader: Limit memory consumption

See merge request GNOME/gtk!4942

jpeg loader: Limit memory consumption

This will prevent stupid oom situations with pathological
jpeg files, without affecting our ability to load reasonable
images.

Merge branch 'jx/hi-res-scrolling' into 'main'

High-resolution scroll wheel support

See merge request GNOME/gtk!3839

ci: Disable -Werror on wayland submodule

CI is mostly interested in GTK not introducing compiler warnings, other
submodules like Wayland might have their own and that shouldn't hinder
CI testing of GTK.

Disable -Werror for the wayland submodule, and let it be fixed independently
at some point.

gtkeventcontrollerscroll: Always accumulate discrete deltas

When GTK_EVENT_CONTROLLER_SCROLL_DISCRETE is set, accumulate deltas also
for mouse scroll so a high-resolution mouse wheel click behaves in the
in the same manner as a low-resolution mouse wheel click.

gdk/x11: Handle high-resolution scroll events

gdk/wayland: Handle high-resolution scroll events

Starting with the Wayland protocol wl_pointer >= 8, discrete axis
events have been deprecated in favour of high-resolution scroll event.

Add a listener for high-resolution scroll events and, for backwards
compatibility, handle discrete events as discrete*120.

gdk/wayland: Calculate discrete scroll direction in its helper

Refactor, no functional changes.

gdk/events: Set discrete scroll deltas in its constructor

Instead of calculating the discrete scroll deltas in
GtkEventControllerScroll, move that code to the event constructor and
access the precalculated values using gdk_scroll_event_get_deltas.

Refactor, no functional changes.

gdk/events: Add constructor for high-resolution scroll events

Starting with Linux Kernel v5.0 two new axes are available for
mice that support high-resolution wheel scrolling: REL_WHEEL_HI_RES and
REL_HWHEEL_HI_RES.

Both axes send data in fractions of 120 where each multiple of 120
amounts to one logical scroll event. Fractions of 120 indicate a wheel
movement less than one detent.

The 120 magic number is a copy of the Windows API, so this new
constructor can be used both in Linux >= 5.0 and Windows >= Vista.

build: Update Wayland to >= 1.21.0

Update Catalan translation

Merge branch 'rounded-clip-node-new-transfer-full' into 'main'

Mark gsk_rounded_clip_node_new() return value as (transfer full)

See merge request GNOME/gtk!4941

Mark gsk_rounded_clip_node_new() return value as (transfer full)

Like all the other constructors for nodes already do. Without this,
newly created rounded clip nodes are leaked in bindings.

treeview: Do not set top row adjustment value while animating

gtk_tree_view_top_row_to_dy, which is called from GtkTreeView's
size_allocate function, changes the adjustment value. Since this
conflicts with the animation when changing the active row, bail
out until the animation is finished.

Fixes #4550

treeview: Use widget coordinates where required

When a GtkTreeView scrolled horizontally, it was not possible to
select rows outside the initial area due to an erroneous comparison
between widget and bin window coordinates.

Original change to widget coordinates occurred in commit
a0de570e47228094f430fcade4e6f56eb75bd8f5

Merge branch 'gbsneto/paned-focus-infloop' into 'main'

paned: Protect against NULL variable

Closes #5094

See merge request GNOME/gtk!4938

Merge branch 'wip/carlosg/fix-crossing-on-unmap' into 'main'

gtkwindow: Use pointer-oriented function to deal with crossing events

Closes #5094

See merge request GNOME/gtk!4937

gtkwindow: Use pointer-oriented function to deal with crossing events

Commit adba0b97 fixed missed pointer crossings by using a helper function that
was already present and looked like did everything that was needed. However
this function was oriented to keyboard focus and it also did update the related
widget state. Doing these changes on pointer-based crossing was misuse, and
could cause weird interactions with keyboard focus management.

Fix this by using gtkmain.c gtk_synthesize_crossing_event() that is in fact
oriented to pointers.

Fixes: adba0b97 (gtkwindow: Synthesize pointer crossing events on state changes)
Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5094

paned: Warn if child is not actually a child

This will at least allow us to debug this in the future.

gtkmain: Pass coordinates when synthesizing pointer events

Instead of passing an event and figuring out coordinates from it, pass
directly the toplevel coordinates so that we can use this outside event
handling.

All callers have been updated to pass the coordinates, in practical effects
they were already based on the GtkNative.

paned: Protect against NULL variable

The inner loop in gtk_paned_set_focus_child() tries to find the
topmost GtkPaned, however, if the `w` variable ends up becoming
NULL after bubbling up the entire GtkWidget hierarchy, this loop
never breaks.

Check for NULL in this loop.

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

Update Indonesian translation

Update Occitan translation

Merge branch 'wip/exalm/multiplier' into 'main'

scrolledwindow: Lower the scroll multiplier

See merge request GNOME/gtk!4933

Updated Spanish translation

scrolledwindow: Lower the scroll multiplier

c68247f63ba1db8849eb1679e53a64f7f2633973 introduced a scroll multiplier,
intended to be significantly lower than the GTK 4.6 behavior but higher
than 1. However, it was _higher_ than 4.6, since 4.6 also had a permanent
1/10 multiplier in GDK, so the cited multiplier values were really 6.4 and
9.7.

Merge branch 'build-fix-glib-2.66' into 'main'

build: fix for use with glib 2.66.0 (2nd try)

See merge request GNOME/gtk!4932

Merge branch 'wip/carlosg/scroll-overshoot' into 'main'

gtk/scrolledwindow: Check for overshoot setting up kinetic scroll helpers

Closes #4784

See merge request GNOME/gtk!4909