Merge branch 'wip/otte/filefilter-name' into 'main'

filefilter: If not given a name, deduce one

See merge request GNOME/gtk!5630

Update Finnish translation

Merge branch 'reverse-file-chooser-recent-files-sort-order' into 'main'

filerchooser: Show Recent files in inverse order

See merge request GNOME/gtk!5625

Merge branch 'wip/otte/listview-border-spacing' into 'main'

Implement CSS border-spacing support for listview and gridview

See merge request GNOME/gtk!5626

Merge branch 'file-chooser-selectability2' into 'main'

file system model: Store selectability

See merge request GNOME/gtk!5582

Merge branch 'notify_after_value_change' into 'main'

a11y: Notify assistive technologies when a GTK_ACCESSIBLE_PROPERTY_VALUE_NOW changes on a widget

See merge request GNOME/gtk!5624

a11y: Notify assistive technologies when a GTK_ACCESSIBLE_PROPERTY_VALUE_NOW changes on a widget

This fixes silent sliders and similar controls when their value is changed.

filefilter: If not given a name, deduce one

Otherwise we end up with a critical when cosntructing the GVariant and
the portal claims our filter is for "[Invalid UTF-8]".

Test included

gridview: Add border-spacing support

omg, this is complicated code.

gridview: Introduce column size functions

This is in preparation for border-spacing integration.

gridview: Use gtk_list_item_manager_get_nearest_tile()

Simplifies gtk_grid_view_get_position_from_allocation() a bit because we
can omit the bounds shenanigan.

listview: Support CSS border-spacing

Hopefully the code can deal with gaps between the tiles.

We use gtk_list_item_manager_get_nearest_tile() when necessary.

filechooser: Set selectability for list items

We propagate this from the list items to the
GtkListItems. Twisty!

filechooser: Dim non-selectable items

columview: Propagate activatable and selectable

When any of the cells is not activatable or selectable,
make the row not have those properties either.

filechooser: Inform the model about selectability

In folder modes, we don't want to select files.

filesystemmodel: Store selectability

This will be used later on.

Merge branch 'file-chooser-selection' into 'main'

filechooser: Don't automatically select a file

See merge request GNOME/gtk!5572

Update Friulian translation

listitemmanager: Add gtk_list_item_manager_get_nearest_tile()

... and make the tile finding code use distance.

This also changes how gtk_list_item_manager_get_tile_at() finds the
right tile, so this is a custom commit for bisectability.

gtk_list_item_manager_get_nearest_tile() isn't used yet.

listview: Move bounds check into base class

This way, listview and gridview don't need to check if the rect is out
of bounds and nothing is selected, a quick rectangle_intersect() does
the job for them.

listview: Return an allocation for unallcoated items

Just get the position right and give them a height of 0px, that should
be good enough.

If we don't do that, code will think the item doesn't exist, which is
not what we want.

filerchooser: Show Recent files in inverse order

When looking at Recent files in the file chooser, it makes more
sense to show the recent files on top.

This commit flips the sort order for that case.

Merge branch 'more-size-allocation-fixes' into 'main'

widget: Skip popovers in allocation

See merge request GNOME/gtk!5615

Merge branch 'wip/chergert/fix-const-preserving-text-iter-order' into 'main'

textlinedisplaycache: fix const-preserving gtk_text_iter_order()

See merge request GNOME/gtk!5620

Merge branch 'file_chooser_stuff' into 'main'

filechooser: two small changes

See merge request GNOME/gtk!5619

textlinedisplaycache: fix const-preserving gtk_text_iter_order()

filechooser: Do not look up parents of directories

If the `GtkRecentInfo` represents a directory, simply use it, and
do not try to find its parent in `_gtk_file_chooser_extract_recent_folders()`.

For example, there is an entry in my recently-used database
from the Amberol music player about the folder I have opened
with it, but the folder is not listed on the "Recent" tab of
the file chooser widget, only its parent. After this change,
the directory itself is shown.

filechooser: Use gtk_recent_info_is_local() for filtering

There is already a function for determining if URI
has the "file" scheme, so use that.

Merge branch 'ebassi/issue-5650' into 'main'

Filter recently used files in the file chooser

Closes #5650

See merge request GNOME/gtk!5618

Merge branch 'fix-msvc-older-glib' into 'main'

modules/media: Fix Visual Studio builds with older GLib (<= 2.74.x)

See merge request GNOME/gtk!5605

widget: Skip popovers in allocation

Native widgets get allocated via their surface,
so can skip them here. This avoids criticals when
re-mapping a popover for the second time, as can
be seen e.g. in the 'Selections' demo in gtk4-demo.

widget:  Stop propagating alloc_needed beyond popovers

This should not be necessary, since popovers get
their new size from present_popup via the compositor.

widget: Remove goto usage in widget_allocate()

widget: Split out gtk_widget_ensure_allocate_on_children()

allocate() should not be calling into ensure_allocate(), they do a similar job.
In the end, the code does the same work, but it should be easier to follow now.

widget: Simplify size allocation

Don't call ensure_allocate if we've just
done size_allocate. This makes criticals
from reshowing popovers go away.

Filter recently used files in the file chooser

Since it's a file selection widget, we should only list recently used
files that point to a file.

Fixes: #5650

Merge branch 'wip/carlosg/reuse-search-engine' into 'main'

gtkfilechooser: Keep GtkSearchEngine around for longer

See merge request GNOME/gtk!5617

gtkfilechooser: Keep GtkSearchEngine around for longer

Currently the GtkSearchEngine is torn down every time the search
is stopped, which also means between typed characters. This
prevents any of the optimizations that the GtkSearchEngine can
do in the long run.

Let the GtkSearchEngine stay around for longer, and only be
disposed after search is cancelled, the filechooser moves
onto a different mode than search, or is otherwise unmapped/disposed.

While at it, remove an unused struct field.

Merge branch 'wip/carlosg/recursive-file-search' into 'main'

searchengine: Improve performance for recursive search

Closes #4133

See merge request GNOME/gtk!5611

searchengine: Do not fight filechooser search mode sorting

Provide the hits in an order that is more easily consumed by the
filechooser, and less jarring when populating.

searchengine: Bail out on the first character

Again on massive filesystems, the very first character
is likely to bring a likewise massive amount of search
results that we need to maybe query info for, then create
icons and widgets for. While it's impressive we can do
that, it's also expensive and likely pointless, for the
first character.

Typing a second character is however very likely to
considerably reduce the amount of items to categorize and
show. So start actually searching from there.

Testing on a filesystem with 1434099 files indexed, trying 5
semi-random 1 character searches (n, h, t, i, o) returns on
average 168K items (min. 78771, max. 331471), trying 5
semi-random 2 character searches (no, he, th, in, on)
returns on average 34K items (min. 11133, max. 94961),
which is a more approachable set.

Doing this is enough that typing on a filechooser search
entry feels completely fluid.

searchengine: Ensure some GFileInfo fields

The search provider should make it sure there are some
specific GFileInfo fields set. Fix the mimetype extraction
from the query, and use that to fill in the missing gaps
the best we can.

searchengine: Drop non-recursive search

Recursive search seems to be the hardcoded default, go with it.

searchengine: Populate filesystem model in an idle

When starting a search over a very populated filesystem, it
is possible that typing the first chars will return a too
high number of results. Even though iterating through the
cursor is in itself very fast, extracting the GIO information
from those many files at once is not going to be as fast.

In order to increase interactivity (i.e. not make things
possibly sluggish) iterate the cursor in an idle function
and add search results to the filechooser model little by little.

If the user keeps typing (as it is likely will happen), there
will be better chances to cancel and proceed to the next
query timely. If not, the results will be there soon enough.

Merge branch 'check_null_gfile_location' into 'main'

filechooser: Fix memory leaks

See merge request GNOME/gtk!5610

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

build: Handle introspection a bit better

See merge request GNOME/gtk!5614

filechooser: Fix memory leaks

The returned strings from `file_chooser_get_location()` were never freed.

build: Handle introspection a bit better

Error out if introspection is requested,
but g-ir-scanner isn't found.

And if introspection isn't explicitly disabled
but is required for building the docs, build it.

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

filechooser: Handle pathless files

See merge request GNOME/gtk!5613

Merge branch 'expose_checkable_state' into 'main'

a11y: When an accessible has GTK_STATE_CHECKED, set its ATSPI_STATE_CHECKABLE as well

See merge request GNOME/gtk!5602

filechooser: Handle pathless files

g_file_get_path() can return NULL. Make our
getter for file locations handle that case.

Related: !5610

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

wayland: Don't crash during DND with Cairo renderer

See merge request GNOME/gtk!5612

wayland: Don't crash during DND with Cairo renderer

Attaching buffers with offsets is forbidden now and nobody
checked the Cairo renderer.

text: Don't crash when somebody drops NULL

Merge branch 'wroy-main-patch-03255' into 'main'

MSBuild: Fix gnome.compile_resources current source directory

See merge request GNOME/gtk!5609

Apply suggestion to use '/' over join_paths

searchengine: Improve performance for recursive search

As fancy as property paths are, recursive resolution of files
to a location increases the big O complexity enough that it's
not a great option on large homedirs with many indexed files.

Ensure the files are from the right location through a URI
prefix match, which does hits an index. This may dramatically
improve performance on large indexed trees.

Testing this query in an isolated testcase with a total
1434099 indexed files shows that it can run more than 1500 times
per second in this computer (an average of 15200 queries in
several 10 second runs), which presumably is a tad faster than
anyone can type.

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

Fix compile_resources present source directory

In certain scenarios, address the issue where gnome.compile_resources
fails to transmit the present source directory. This is most notably
visible with MSBuild.

Merge branch 'unbreak-combobox' into 'main'

combobox: Avoid extra queue_resize()

Closes #5644

See merge request GNOME/gtk!5608

Updated Danish translation

combobox: Avoid extra queue_resize()

width-request already ensures it's above the minimum width, so avoid an
extra queue_resize() when setting size request to (-1, -1).

This is the same way as GtkDropDown works. This also unbreaks
GtkComboBox after the recent allocation fix in
75a417e33708dab2bdb2f784a8952e085a12bf03.

Incidentally, this also makes GtkComboBox actually resize its popup as
intended (that was broken before).

I don't think this is ultimately the final fix, sometimes I still get
allocation warnings. But the proper fix will probably involve changing
some more allocation machinery around popovers. This is good enough for
now.

Merge branch 'window-signals' into 'main'

window: Disconnect the ::compute-size handler

See merge request GNOME/gtk!5606

window: Disconnect the ::compute-size handler

All the other signal handlers are connected in
realize and disconnected in unrealize, but the
::compute-size handler was forgotten.

This was notices in !5597.

modules/media: Fix Visual Studio builds with older GLib

The current definitions of the g_io_module_*() symbols do not build on
Visual Studio when building against GLib earlier than 2.75.0 due to the
way how these symbols are decorated in the GLib headers, as Visual Studio
does not allow symbols that were previously marked with 'extern' (or so)
to be marked with anything that is symantically different later.

As a result, if we are using Visual Studio and glib-2.74.x or earlier,
override _GLIB_EXTERN as appropriate in the modules/media sources before
including the GIO headers.  This sadly, means that we need a
configure-time check as it would have been too late if we checked the
GLib version using G_VERSION_CHECK macro, as the GIO headers would have
been included already.

There are similar items in the print backends, but we will not attempt
to update these files as they are not meant to be built for Windows.

Merge branch 'expose_landmark_role' into 'main'

a11y: Expose GTK_ACCESSIBLE_ROLE_LANDMARK to AtSPI2

See merge request GNOME/gtk!5601

a11y: When an accessible has GTK_STATE_CHECKED, set its ATSPI_STATE_CHECKABLE as well

This allows, for example, checkable list itemss.

a11y: Expose GTK_ACCESSIBLE_ROLE_LANDMARK to AtSPI2

Update Czech translation

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

widget: Use the correct template in dispose_template()

See merge request GNOME/gtk!5600

Merge branch 'main' into 'main'

removed duplicate function call

See merge request GNOME/gtk!5545

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

macos: Fix type func generation

See merge request GNOME/gtk!5596

Merge branch 'wip/fix-wl-destruct-order' into 'main'

wayland: Destroy xdg_surface after role

See merge request GNOME/gtk!5598

widget: Use the correct template in dispose_template()

In derivable classes, the widget's class can be different from the one
dispose_template() was called for, which can lead to failing the
template != NULL check at best, undefined behavior at worst.

Since we already pass the correct GType into the function, just use that
instead.

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

inspector: Make really sure we don't inspect ourselves

See merge request GNOME/gtk!5599

inspector: Make really sure we don't inspect ourselves

The problem here is that new windows appear in the list before the
window's dispay gets set and we don't update the filter when the
display changes (would need watches support for the filtermodel).

So add this somewhat hacky method.

wayland: Destroy xdg_surface after role

The split-up of gdksurface-wayland.c introduced a protocol violation
when it didn't make sure xdg_surface was destroyed after the role
objects (xdg_popup / xdg_toplevel). Fix that.

Fixes: 2a463baed0 ("wayland: Rearrange the surface code")

macos: Fix type func generation

We were checking for gdk_quartz even though
the sysbols are now all called gdk_macos. Oops.

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

cssnode: Don't crash the inspector

See merge request GNOME/gtk!5595

testsuite: Add tests for the unknown enums

See commit 40e7a265a7ccb1385a3af0931c591201b0ce063a

testsuite: Add empty test for new node

cssnode: Don't crash the inspector

Don't misinform the observing listmodel that CSS nodes were removed that
weren't actually removed, but just moved. Otherwise the observer would
think it has run out of items when it really hasn't.

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

meson: Mention testsuite in summary

See merge request GNOME/gtk!5594

meson: Mention testsuite in summary

This was broken out from the 'tests' toggle,
so list it separately.

Merge branch 'expose_error_message_relation' into 'main'

a11y: Expose GTK_ACCESSIBLE_RELATION_ERROR_MESSAGE to AtSPI2

See merge request GNOME/gtk!5588

Merge branch 'wip/otte/listview2' into 'main'

listview: GtkListTile

See merge request GNOME/gtk!5584

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

ci: Disable the testsuite in flatpak builds

See merge request GNOME/gtk!5593

ci: Disable the testsuite in flatpak builds

No need to build the testsuite when we are
just producing flatpaks.

listbase: Remove nonexisting function from header

gridview: Add a filler tile for empty space

That stupid space in the bottom right when n_items isn't a multiple of
n_columns needs its own tile, or we'll get errors about not finding a
tile.

So make one.

columnview: Clear the sorter first thing in dispose

Otherwise, when removing the columns, each column will trigger a
sorter::changed signal emission.

And because sorters are often still connected to a sortlistmodel, we
can't skip that emission and need to do it.
But we only need to do it once.

listview: Handle emptying of views

The previous check does not longer work.

When a model gets all items deleted, there will still be existing tiles
until the next time garbage collection is run.
So do that before checking if the list is empty.

listview: Remove an unused member variable

It's only used in the allocate function, so make it a local varaible
there.

listview: Simplify a vfunc

Instead of making it 2 vfuncs for getting horizontal and vertical area,
make it one vfunc to get the area.

Also rewrite the implementations to use the tile's area instead of
trying to deduce things with fancy math.

listview: Simplify allocation

With the Tile changes, a lot of stuff does no longer need to be
duplicated between listview and gridview. Move it to ListBase instead.

gridview: Remove an unused member variable

It's only used during size_allocate(), so make it a local variable
there.

listitemmanager: Add a split vfunc and use it

Instead of randomly changing tiles, the listitemmanager gains a split
vfunc that listview and gridview implement so they can keep their tile
areas intact. The listitemmanager will now conform to these rules:

1. Never delete a tile.
   This ensures that all areas stay intact.

2. Never change the n_items of a tile other than setting them to 0.
   This causes "empty" areas to appear, but listview/gridview can
   easily check for them by checking for tile->n_items == 0.
   gtk_list_tile_gc() will get rid of them.

3. Adding items always creates new tiles that are added with empty area.
   That way they don't interrupt any existing machinery until the next
   allocation.

4. Adding/removing widgets has no effect on areas
   This is useful in particular when scrolling where new widgets are
   moving between tiles. When the manager moves the widgets, it may
   split some areas, but will not remove any existing tiles, so the
   whole area stays intact and the list can deal with further scroll
   events before an allocation.

This improve the situation for #3334

listview: Add gc'ing

This is in preparation for the following reorg of the listitemmanager,
it should not have any effect now.

listitemmanager: Remove unused functionality

Both ListView and GridView use GtkListTile now, so no need anymore for
custom machinery.

gridview: Redo tile management

Instead of the custom size property, use the new tile size.

Also introduce the ability to split tiles, so that gridview can split a
layout that would look like (question mark denoting cells without a
widget, which in this case would be a single tile)

█ █ █ ? ?
? ? ? ? ?
? ? ? ? ?
? ? ?

into 3 rectangular tiles like so:

█ █ █ A A
B B B B B
B B B B B
C C C

This of course also means we need to be able to merge those tiles again
when cells got added/deleted or the gridview was resized. For that job,
gtk_list_tile_gc() exists now, which removes tiles without items and
merges adjacent tiles without widgets.