widget: Shortcut widget not repositioning

If a widget doesn't move positions during gtk_widget_reposition(),
detect it early.

This is especially beneficial in the inspector.

Update Bulgarian translation

(cherry picked from commit 3458ebf8c93f839cbf33caa48cf39ec112f292a2)

Merge branch 'more-property-notification' into 'main'

filedialog: Make filters nullable

See merge request GNOME/gtk!5755

testsuite: Check more property notification

Handle more value types.

Make adjustment setters behave

models: Fix up property notifications

We were emitting redundant notifications
for the ::model property.

filedialog: Make filters nullable

It should be possible to reset a property
to its default value.

Merge branch 'more-notify-tests' into 'main'

testsuite: Clean up some exclusions

See merge request GNOME/gtk!5752

Merge branch 'gbsneto/wayland-buffer-scale-creation' into 'main'

gdk/wayland/surface: Track and apply buffer scale

See merge request GNOME/gtk!5751

Make widget property setters behave

We want to allow setting a property to its
current value, while still having a precondition
for widget->parent being NULL otherwise.

gdk/wayland/surface: Track and apply buffer scale

Add a new flag to track whether buffer scale is dirty or not,
and centralize calling wl_surface_set_buffer_scale() in a single
place: gdk_wayland_surface_sync_buffer_scale().

gdk_wayland_surface_sync_buffer_scale() is only called by
gdk_wayland_surface_sync(), which itself is called by the GL,
Vulkan, and Cairo contexts, right before submitting a frame.
This ensure that each frame has an up-to-date buffer scale.

This mimics how opaque and input regions are tracked.

testsuite: Check more property notification

We can easily check some object types
such as widgets.

testsuite: Clean up some exclusions

Drop some exclusions from the notify test.
The tests work fine for these objects.

listitem: Fix a notification problem

We were notifying ::item when ::child is
changed. Oops

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

emojichooser: Stop using gtk_widget_get_allocation

See merge request GNOME/gtk!5749

emojichooser: Stop using gtk_widget_get_allocation

Replace all uses with gtk_widget_compute_bounds.

Merge branch 'fix-popover-positioning' into 'main'

wayland: Fix corner cases in positioning

See merge request GNOME/gtk!5741

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

Add some more keysyms tests

See merge request GNOME/gtk!5745

Add some more keysyms tests

Spot-check a few keypad symbols.

testsuite: Update a test

cacecd2fb54025a changed a warning message that
some tests check for. Oops.

Merge branch 'wip/otte/filechooser-fixes' into 'main'

filechooser fixes

See merge request GNOME/gtk!5746

Merge branch 'wip/otte/node-naming' into 'main'

node format: Allow naming textures and nodes

See merge request GNOME/gtk!5740

filechoosercell: Remove position property

Instead, query the position from the listitem

filechoosercell: Remove selected property

Instead, query the selected state from the listitem.

filechooser: Select via action

Don't fudge around poking through the listview, trying to get a model
and selecting it directly. Instead, use the proper way and activate the
"listitem.select" action.

listbase: Select via action

Instead of directly calling select_item(), trigger the select-item
action of the focused child.

We do this convoluted calling into the widget because that way
GtkListItem::selectable gets respected, which is what one would expect.

Plus, this code is usually triggered via keybindings, and this way the
ListBase keybindings work identical to the ListItem keybindings.

Merge branch 'wip/otte/trelistmodel-fixage' into 'main'

treelistmodel: Don't add items in reverse

Closes #5707

See merge request GNOME/gtk!5744

docs: Document additions to the node format

testsuite: Add parsing tests for node and texture naming

rendernodeparser: Detect duplicates and print them by name

If we encounter a node or texture the 1st time and they are going
to be used again, give them a name.

Then, when encountering them again, print them by name instead
of duplicating them.

rendernodeparser: Add support for reusing nodes

We extend the syntax for nodes from:
  <node-type> { ... }
to
  <node-type> { ... }
  <node-type> <string> { ... }
  <string>;
where the first is the same as before, the 2nd defines a named node and
the last references a previously defined node.
Or to give an example:
  color "node" {
    bounds: 0 0 10 10;
    color: red;
  }
  transform {
    bounds: 20 0 10 10;
    child: "node";
  }
This will draw the red box twice, once at (0,0) and once at
(20,0).

The intended use for this is both shortening generated node files as
well as allowing to write tests that reuse nodes, in particular when
dealing with caches.

rendernodeparser: Add support for reusing textures

We extend the syntax for textures from just:
  <url>
to
  [<string>] <url>
  <string>
where the first defines a named texture while the second references a
texture.
Or to give an example:
  texture {
    bounds: 0 0 10 10;
    texture: "foo" url("foo.png");
  }
  texture {
    bounds: 20 0 10 10;
    texture: "foo";
  }
This will draw the texture "foo.png" twice, once at (0,0) and once at
(20,0).

The intended use for this is both shortening generated node files as
well as allowing to write tests that reuse textures, in particular when
mixing them in texture and texture-scale nodes.

rendernodeparser: Add a Context object

... and pass it through all the parsing functions.

So far it's unused, this is just the plumbing.

rendernodeparser: Split out a function

This code was copy/pasted, so unify it.

rendernodeparser: Emit error when failing to resolve URL

rendernodeparser: Failing to parse a string is an error

Testcase included.

testsuite: Fix broken test

The test was referencing a nonexisting image.

Plus, node parsing cannot reference relative URLs.

Add a test for splicing treelistmodel

treelistmodel: Don't add items in reverse

This was broken since commit bf55685000fd744a26eec61c07fd8aa3d084d54c
where we started to track items.

Fixes #5707

wayland: Fix corner cases in positioning

We were not handling some cases correctly.
This was making the dropdown in the gtk-demo
image-scaling demo appear in the top left
corner.

Merge branch 'wayland-popup-anchor' into 'main'

wayland: Comply with protocol requirements

See merge request GNOME/gtk!5723

Update French translation

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

gdk: Improve logging for backends

Closes #5704

See merge request GNOME/gtk!5737

Merge branch 'sumibi-yakitori/fix-texture-filter' into 'main'

glrenderer: Fix to prevent creation of GL_TEXTURE_MAG_FILTER filters with incorrect values

See merge request GNOME/gtk!5729

gdk: Avoid a pointless indirection

Don't use g_fprintf if GLib is using system printf
anyway.

docs: Some tweaks

Improve an error message

We were producing a misleading error message
when gtk_init() fails.

Fixes: #5704

gdk: Improve logging for backends

Log more when trying backends.

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

clenaup

See merge request GNOME/gtk!5736

columnviewcell: Remove root/unroot stuff

Instead, do that work on creation/dispose.

This is a followup to 3fb10ff2d8cdffdc5e026f9b57c4d5580a36d308.

columnviewcell: Remove "visible" property setting

Since 7ad693f865fc7ca36b63301ef261e3030f321d47 we only create cells for
visible rows and don't just toggle visibility.

This is a leftover from that change.

enums: Move G_END_DECLS to the end of the file

Merge branch 'bilelmoussaoui/gi-docs' into 'main'

docs: Fix various broken links

See merge request GNOME/gtk!5734

Merge branch 'wip/reset-repositioned-after-handling' into 'main'

wayland/popup: Handle late received reposition feedback

See merge request GNOME/gtk!5735

docs: Fix various broken links

wayland/popup: Handle late received reposition feedback

If we map, reposition, unmap, remap, the reposition feedback from the
last time a popup was mapped might be received while we're dealing with
the new version of the popup. At this point, the old reposition token
has no meating, so lets drop it. Also reset the reposition tokens when
creating new protocol objects, so that the reposition token are as if
we're in the initial state.

This fixes an issue where we'd get stuck if repeatedly smashing a button
that'd create popups that'd immediately get dismissed by the compositor.

Merge branch 'ebassi/compute-size-annotation' into 'main'

Remove wrong annotation for GdkToplevel::compute-size

See merge request GNOME/gtk!5725

Merge branch 'wayland_display' into 'main'

wayland: Try to open the display even if XDG_RUNTIME_DIR is unset

See merge request GNOME/gtk!5732

wayland: Try to open the display even if XDG_RUNTIME_DIR is unset

Since Wayland 1.15, it is now possible to use absolute paths in
"WAYLAND_DISPLAY".
In that scenario, having a valid "XDG_RUNTIME_DIR" is not a requirement
anymore.

For this reason we remove the "XDG_RUNTIME_DIR" check and we let
`wl_display_connect()` decide if our environment is correct.

Signed-off-by: Ludovico de Nittis <ludovico.denittis@collabora.com>

Merge branch 'wip/otte/listview-inert' into 'main'

listview: Add concept of inertness

See merge request GNOME/gtk!5731

gridview: Add concept of inertness

An inert gridview is a gridview that does not use the factory. This
allows faster updates because no calls into user code need to happen.

A gridview is inert when either:
 - It is not rooted.
 - It is not visible.
 - No factory is set (that one is obvious)

The gridview does not need to be inert without a model, as that case is
handled by the item manager.

This should allow Nautilus to keep both the gridview and the columnview
around, and just gtk_widget_hide() the unused widget.

The code for now does not disable the item manager, as some
functionality of the item manager is required to allow setting scroll
positions and such.
But that is a place where more gains could be found if profiling showed
that was useful to do.

listview: Add concept of inertness

An inert listview is a listview that does not use the factory. This
allows faster updates because no calls into user code need to happen.

A listview is inert when either:
 - It is not rooted.
 - It is not visible.
 - No factory is set (that one is obvious)

The listview does not need to be inert without a model, as that case is
handled by the item manager.

This should allow Nautilus to keep both the gridview and the columnview
around, and just gtk_widget_hide() the unused widget.

The code for now does not disable the item manager, as some
functionality of the item manager is required to allow setting scroll
positions and such.
But that is a place where more gains could be found if profiling showed
that was useful to do.

listfactorywidget: Stop trying to be smart

The widget would teardown the factory on unroot to avoid unnecessary
work when it isn't shown.

However, recycling may reposition widgets, and repositioning widgets
does a unroot/root.

We don't want to recycle widgets then.

gtk-demo: Set tab behavior for settings demo

magnifier: Don't check for visibility

gtk_widget_queue_draw() already does that.

glrenderer: Fix to prevent creation of GL_TEXTURE_MAG_FILTER filters with incorrect values

Merge branch 'wip/otte/tab-behavior' into 'main'

listview: Add ::tab-behavior to list widgets

See merge request GNOME/gtk!5710

columnview: Add GtkColumnView::tab-behavior

Also make sure that GTK_LIST_TAB_CELL works by implementing support for
it in the row widget.

gridview: Add GtkGridView::tab-behavior

listview: Add GtkListView::tab-behavior

The implementation lives (as always) in GtkListBase.

This is a feature request from the Nautilus developers, who currently do
some hacks to emulate that behavior and it apparently only breaks
sometimes.

Merge branch 'scrolled-window-signal-disconnnect' into 'main'

scrolledwindow: Disconnect the right handler

Closes #5684

See merge request GNOME/gtk!5726

scrolledwindow: Disconnect the right handler

We connect gtk_scrolled_window_update_use_indicators
as signal handler in realize(), but we were disconnecting
gtk_scrolled_window_sync_use_indicators in unrealize.

Spotted by Milan Crha.

Fixes: #5684

Remove wrong annotation from GdkDragSurface::compute-size

Just like GdkToplevel::compute-size, the size argument of the signal is
given to the handlers by GDK; it's not an out argument meant to be
allocated by the caller.

Remove wrong annotation for GdkToplevel::compute-size

The size argument is passed to the signal by the GDK surface machinery,
as is: it's not going to be allocated by the caller (since it's a
signal), and it's not an out argument.

Merge branch 'wayland-startup-notification-crash' into 'main'

wayland: Don't crash without xdg_activation_v1

Closes #5701

See merge request GNOME/gtk!5724

wayland: Don't crash without xdg_activation_v1

If the compositor does not support this protocol,
we can't use it.

Fixes: #5701

wayland: Comply with protocol requirements

If you send a bad anchor rect to mutter, it crashes.
Thats not great, so lets not do that.

Merge branch 'wayland-cursor-size' into 'main'

wayland: Don't crash on cursor size 0

Closes #5700

See merge request GNOME/gtk!5721

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

popover: Fix typo

See merge request GNOME/gtk!5720

wayland: Don't crash on cursor size 0

The cursor-theme-size setting is documented as
'0 means the default size'. Make it so by using
size 24 if we see a 0. Its better than crashing.

Fixes: #5700

gtk-demo: Rename dropdown demo

It's the listview selections demo, so use that name.

gtk-demo: Rename image scaling demo to image_scaling

This just means renaming the C file, but it makes --run=image_scaling
actually do what I'd expect.

popover: Fix typo

The typo causs gnome-shell to crah, which is somewhat suboptimal.

It was introduced in 1a9c7a4b1fb.

Merge branch 'coordinate-docs' into 'main'

docs: Add some cross references

See merge request GNOME/gtk!5719

Incorporate review feedback

docs: Use our own box model illustration

This one has a dark mode too.

Tweak

Tweak

More review feedback

docs: Add another cross reference

Refer to the new coordinate system overview
from the GdkEvent docs.

docs: Apply review feedback

Clarify some points in the coordinate overview.

Docs: Cosmetics

Align section heading better. Now they all follow
the "Using GTK..." pattern.

docs: Add some cross references

Refer to the new coordinate system overview
from a couple of places where it is useful.

docs: Add a section about coordinate systems

Merge branch 'deprecate-translate-coordinates' into 'main'

Deprecate gtk_widget_translate_coordinates

Closes #5697

See merge request GNOME/gtk!5717

Drop gtk_widget_get_surface_allocation

This private api is confusing, and no longer used.

entrycompletion: Stop using gtk_widget_get_surface_allocation

We only need the width of the widgets' bounds
here anyway, so just use gtk_widget_compute_bounds.

popover: Stop using gtk_widget_get_surface_allocation

The function is going away, and the computation
here was wrong anyway. Instead, add a helper that
properly computes the pointing-to rect in surface
coordinates and use it everywhere.

Deprecate gtk_widget_translate_coordinates

Update all (non-deprecated) callers to use
gtk_widget_compute_point instead.

Fixes: #5697

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

testsuite: Add some GL texture tests

See merge request GNOME/gtk!5715