pathbar: Special case recent button

Soon pathbar will be shown on the Recent mode, with a static
path pointing to, well, Recent, so special case it in the
GtkPathBar with a static icon.

filechooserwidget: Hide ColumnView settings in GridView

filechooser: Save and restore view type

With this, the filechooser comes up with the
same view (grid or list) that it had the last
time you used it.

filechooserwidget: Rename callback

It's shared between both column and grid views now.

filechooserwidget: Implement 'keynav-failed' for grid view

It's exactly the same as of the column view. Since we're sharing
the callback now, rename it to browse_files_view_keynav_failed_cb.

filechooserwidget: Use correct view in various places

Most of the pointer comparisons against 'browse_files_column_view'
should actually be performed against the current view widget. As
it turns out, it weren't that many places after all.

style: Improve filechooser grid view slightly

filechooserwidget: Add a grid view

Add a grid view outside of the widgetry tree. The grid view mimics
the column view using bindings, so we only need to manage the column
view.

Also add a button in the path bar section to toggle the view. This
is handled as a new 'toggle-view' action in the file chooser.

The way switching between views currently work is by setting either
the column or grid view as the child of the GtkScrolledWindow. This
has the benefit of unmapping the unused view, which is nice and can
avoid some tricky situations with thumbnails.

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

filethumbnail: Add 'icon-size' property

This will be used by future commits so that we can reuse this widget
in the file chooser grid view.

Merge branch 'harfbuzz-dep' into 'main'

build: Use a stable harfbuzz

See merge request GNOME/gtk!5327

build: Use a stable harfbuzz

We don't need the latest harfbuzz.
Stick to 4.0 for now.

Merge branch 'docs/default-args' into 'main'

docs: Emphasize requirement to provide filenames as arguments

See merge request GNOME/gtk!5322

docs: Emphasize requirement to give filenames as arguments

(related to #5430)

Merge branch 'sunflowerskater-main-patch-64457' into 'main'

appchooser: Rename "application" to "app"

See merge request GNOME/gtk!5323

appchooserbutton: Rename "application" to "app"

This change is a part of an initiative to use the word "app" instead of "application" in the UI:

Teams/Design/initiatives#123

appchooserwidget: Rename "application" to "app"

This change is a part of an initiative to use the word "app" instead of "application" in the UI:

https://gitlab.gnome.org/Teams/Design/initiatives/-/issues/123

appchooser: Rename "application" to "app"

This change is a part of an initiative to use the word "app" instead of "application" in the UI: https://gitlab.gnome.org/Teams/Design/initiatives/-/issues/123

Merge branch 'popover-present-docs' into 'main'

popover: Clarify docs

See merge request GNOME/gtk!5318

Merge branch 'ebassi/image-dispose' into 'main'

Do not change the notification queue inside finalization

Closes #5420

See merge request GNOME/gtk!5321

Merge branch 'builder-scope-note' into 'main'

builderscope: Add a note for implementors

See merge request GNOME/gtk!5317

Do not change the notification queue inside finalization

Notifications are not going to be emitted during the finalization, and
GObject will warn if you try to acquire and release a reference to the
notification queue when the reference count of an object has reached
zero.

Fixes: #5420

builderscope: Add a note for implementors

Recommend that scope implementations should fall
back to or derive from GtkBuilderCScope in order
to not lose GTK's type-guessing machinery.

Related: #5398

popover: Clarify docs

gtk_popover_present() is a misleadingly named
function, so at least give it clear docs that
tell people to use gtk_popover_popup() instead.

Merge branch 'docs/deprecated-G_APPLICATION_FLAGS_NONE' into 'main'

docs: Update deprecated G_APPLICATION_FLAGS_NONE in examples

See merge request GNOME/gtk!5315

docs: Update deprecated G_APPLICATION_FLAGS_NONE

Merge branch 'deprecate-keys-changed' into 'main'

treeview: Drop a few unused variables

See merge request GNOME/gtk!5314

window: Deprecate the ::keys-changed signal

tests: Silence a compiler warning

treeview: Drop a few unused variables

Compilers get pickier every year...

composetable: Drop an unused variable

Add Interlingue translation

Merge branch 'dialog-fixes' into 'main'

filechooserwidget: Fixes

See merge request GNOME/gtk!5311

openuriportal: Better error handling

Nested async calls are always a challenge.

Hopefully, things are straightened out now,
and we report GTK_DIALOG_ERROR errors for
the cases we care about.

openuriportal: Add a check function

Add private API to check whether the portal
is available.

openuriportal: Support OpenDirectory

This required an extra argument. Update all callers.

openuriportal: Cosmetics

There were some copy-paste errors.

fontdialog: Improve docs

Be more specific about possible errors.

colordialog: Improve the docs

Be more specific about possible errors.

fontdialog: Don't forget to propagate errors

Oops.

fontdialogbutton: Survive window closing

If the parent window of the button gets destroyed
while the dialog is open, we cancel the async op,
but we need to be a little more careful about not
stepping on glass.

colordialogbutton: Survive window closing

If the parent window of the button gets destroyed
while the dialog is open, we cancel the async op,
but we need to be a little more careful about not
stepping on glass.

fontdialog: Unset check cancellable

colordialog: Unset check cancellable

filedialog: Report cancellation propertly

Unless we prevent it, GTask will always report
cancellation as G_IO_ERROR_CANCELLED. But we
want GTK_DIALOG_ERROR_ABORTED for this case.

filechooserwidget: Fixes

Fix a memory leak in the file manager launch path.

Merge branch 'wip/corey/column-view-header' into 'main'

columnviewtitle: Prevent header clicks from propagating

Closes #5425

See merge request GNOME/gtk!5306

Merge branch 'matthiasc/node-editor-srcdir' into 'main'

node-editor: Redo saving location

Closes #5403

See merge request GNOME/gtk!5310

Remove leftover debug spew

node-editor: Redo saving location

Determine the location to save testcases in dynamically,
trying first a GTK_SOURCE_DIR environment variable
and then the current directory as the GTK source dir,
ultimatively falling back to just saving in the current
directory.

This avoids leaking details of the build environment
into the produced artifacts and should make GTK builds
more reproducible.

Fixes: #5403

Merge branch 'fix-file-dialog-title' into 'main'

filedialog: Use title if set

Closes #5415

See merge request GNOME/gtk!5308

filedialog: Use title if set

This broken when the code was reshuffled into
the switch.

Fixes: #5415

Merge branch 'wip/carlosg/stale-timeout' into 'main'

gdk/x11: Clear all data in GdkSurfaceX11 finalization

See merge request GNOME/gtk!5303

Merge branch 'macos-unused-vars' into 'main'

macos: remove unused variables from macos code

See merge request GNOME/gtk!5290

Merge branch 'ccook/spelling-changes' into 'main'

Various spelling mistakes across a few domains that show up in documentation.

See merge request GNOME/gtk!5292

Various spelling mistakes across a few domains that show up in documentation.

Merge branch 'emoji-recent-empty' into 'main'

emojichooser: Actually disable the recent section

See merge request GNOME/gtk!5305

columnviewtitle: Prevent header clicks from propagating

The header in GtkColumnView has multiple event handlers
there is a ::pressed handler in GtkColumnView for
resizing the columns in CAPTURE as well as a motion
and drag controller.  The ::release handler is in
GtkColumnViewTitle.   We can't claim the event in the
existing handlers because then the ::release handler will
never get called.  Currently, however, all clicks get propagated
to the ColumnView from the header which can be problematic.
Since we don't usually want the clicks from the header
handled on the view, claim it in the BUBBLE phase.

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

emojichooser: Actually disable the recent section

The loop sets empty = FALSE when there are emojis but for that
to work we need to initialize the value to TRUE initially.

Fixes: 7928532bc5c33b2c6a10bae138ebc6dca1843cb0

gdk/x11: Clear all data in GdkSurfaceX11 finalization

Currently, the GdkSurfaceX11 implementation relies that the upper
layers hid the surface before destruction, and that no
GdkSurfaceClass.compute_resize happened between them. If these
circumstances happened, there would be a compute_size timeout left
dangling after the surface got destroyed, poking at incorrect data
later on. Something that looks like this was reported in the
recent mutter-x11-frames "SSD frames server":

    mutter-x11-frames:423016): GLib-GObject-WARNING **: 19:41:16.869: invalid unclassed pointer in cast to 'GtkWindow'

    Thread 1 "mutter-x11-fram" received signal SIGTRAP, Trace/breakpoint trap.
    g_logv (log_domain=0x7ffff7f7c4f8 "GLib-GObject", log_level=G_LOG_LEVEL_WARNING, format=<optimized out>, args=<optimized out>) at ../../../glib/gmessages.c:1433
    1433 ../../../glib/gmessages.c: No such file or directory.
    (gdb) bt
    #0  g_logv (log_domain=0x7ffff7f7c4f8 "GLib-GObject", log_level=G_LOG_LEVEL_WARNING, format=<optimized out>, args=<optimized out>) at ../../../glib/gmessages.c:1433
    #1  0x00007ffff73470ff in g_log (log_domain=log_domain@entry=0x7ffff7f7c4f8 "GLib-GObject", log_level=log_level@entry=G_LOG_LEVEL_WARNING, format=format@entry=0x7ffff7f84da8 "invalid unclassed pointer in cast to '%s'")
        at ../../../glib/gmessages.c:1471
    #2  0x00007ffff7f72892 in g_type_check_instance_cast (type_instance=type_instance@entry=0x5555558e04b0, iface_type=<optimized out>) at ../../../gobject/gtype.c:4144
    #3  0x00007ffff791e77d in toplevel_compute_size (toplevel=<optimized out>, size=0x7fffffffe170, widget=0x5555558e04b0) at ../../../gtk/gtkwindow.c:4227
    #4  0x00007ffff7f4f3b0 in g_closure_invoke (closure=0x555555898cc0, return_value=return_value@entry=0x0, n_param_values=2, param_values=param_values@entry=0x7fffffffdeb0, invocation_hint=invocation_hint@entry=0x7fffffffde30)
        at ../../../gobject/gclosure.c:832
    #5  0x00007ffff7f62076 in signal_emit_unlocked_R
        (node=node@entry=0x55555588feb0, detail=detail@entry=0, instance=instance@entry=0x55555560e990, emission_return=emission_return@entry=0x0, instance_and_params=instance_and_params@entry=0x7fffffffdeb0)
        at ../../../gobject/gsignal.c:3796
    #6  0x00007ffff7f68bf5 in g_signal_emit_valist (instance=<optimized out>, signal_id=<optimized out>, detail=<optimized out>, var_args=var_args@entry=0x7fffffffe050) at ../../../gobject/gsignal.c:3549
    #7  0x00007ffff7f68dbf in <emit signal ??? on instance 0x55555560e990 [GdkX11Toplevel]> (instance=<optimized out>, signal_id=<optimized out>, detail=detail@entry=0) at ../../../gobject/gsignal.c:3606
    #8  0x00007ffff7a8de96 in gdk_toplevel_notify_compute_size (toplevel=<optimized out>, size=size@entry=0x7fffffffe170) at ../../../gdk/gdktoplevel.c:112
    #9  0x00007ffff7a4b15a in compute_toplevel_size (surface=surface@entry=0x55555560e990 [GdkX11Toplevel], update_geometry=update_geometry@entry=1, width=width@entry=0x7fffffffe220, height=height@entry=0x7fffffffe224)
        at ../../../gdk/x11/gdksurface-x11.c:281
    #10 0x00007ffff7a4c3b2 in compute_size_idle (user_data=0x55555560e990) at ../../../gdk/x11/gdksurface-x11.c:356
    #11 0x00007ffff733f67f in g_main_dispatch (context=0x55555563f6e0) at ../../../glib/gmain.c:3444
    #12 g_main_context_dispatch (context=context@entry=0x55555563f6e0) at ../../../glib/gmain.c:4162
    #13 0x00007ffff733fa38 in g_main_context_iterate (context=0x55555563f6e0, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at ../../../glib/gmain.c:4238
    #14 0x00007ffff733fcef in g_main_loop_run (loop=loop@entry=0x5555560874a0) at ../../../glib/gmain.c:4438
    #15 0x0000555555557de0 in main (argc=<optimized out>, argv=<optimized out>) at ../src/frames/main.c:68

It perhaps makes sense to warn in these situations, but either way
it sounds like gdk_surface_x11_finalize() could enforce the correct
behavior by ensuring there is no dangling timeouts/data. This commit
does that.

Cleanup: remove unused variables from macos code

Remove unused variables. In case if Cairo code it would
allocate memory, which would result in a memory leak.

Merge branch 'workaround-glx-issue' into 'main'

GLX: Add extra glXMakeContextCurrent () call to work around a DRI issue

Closes #4499, #5387, and #5170

See merge request GNOME/gtk!5285

GLX: Add extra glXMakeContextCurrent () call to work around a DRI issue

Reference: https://gitlab.freedesktop.org/mesa/mesa/-/issues/7808

Fixes #4499, #5387, #5170

Merge branch 'wip/corey/tree-expander-bug' into 'main'

Improve treeexpanders

See merge request GNOME/gtk!5282

list: Remove active state handling in listitemwidget and treeexpander

Active state is handled by main now.  It appears that listitem and
treeexpander handled it manually (probably before main did).  This
is unnecessary now, so let's remove it.

treeexpander: Scroll to expanded item

If the anchor is below the expanded item, the expanded item will
go out of view if there are sufficient children items.  This is not
ideal, so make sure to scroll to the item to ensure it remains in
view.

listitemwidget: Add listitem.scroll-to action

This is the matching action for list.scroll-to-item

treeexpander: Toggle expand on release instead of pressed

With the introduction of the hide-expander property, I noticed that
the active state would persist in many cases because the release
signal was never emitted.  In gtk3 tree expanders, expanded
on release.  gtk4 expanded on press to match window explorer.
Per irc chat, the designers didn't have a strong preference
for press or release.  In order to keep consistency and
fix this bug, let's move back to release.

treeexpander: Fix typo and GestureClick type

"canceled" should be "cancelled".  Also, "cancel" is a signal on
GtkGesture.

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

docs: Fix property cross-links in GtkTreeExpander

See merge request GNOME/gtk!5279

docs: Fix property cross-links in GtkTreeExpander

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

testsuite: Skip tests that miss charset support

See merge request GNOME/gtk!5277

Merge branch 'ccook/spelling-fixes' into 'main'

docfix: Various spelling errors that show up in docs

See merge request GNOME/gtk!5272

testsuite: Skip tests that miss charset support

This came up when running the tests in a fresh
toolbox missing the glibc-gconv-extra package.

Merge branch 'wip/corey/at-spi-context' into 'main'

atspicontext: Cache empty atspi bus address

See merge request GNOME/gtk!5276

Merge branch 'show-hide-cleanup' into 'main'

gtk: Stop using gtk_widget_show/hide

See merge request GNOME/gtk!5273

atspicontext: Log more helpful warnings when the a11y bus doesn't exist

atspicontext: Cache empty atspi bus address

We are caching the bus address as data on the display object when it
exists, but fail to set the data when the bus address doesn't exist.
That causing excessive calls to GetAddress when the accesssbility
bus doesn't exist. Make sure to cache a non-existent accessibility
bus by setting the "" string.

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

Revert "builder: Stop using GtkBuildable.get_id"

See merge request GNOME/gtk!5274

Merge branch 'wip/carlosg/osk-activation-in-widgets' into 'main'

Handle OSK activation in GtkText/GtkTextView widgets

See merge request GNOME/gtk!5269

Revert "builder: Stop using GtkBuildable.get_id"

This reverts commit 92edd4c476917e83c5f50e5eab8aa5a0ad240793.

This commit changed behavior that GNOME Builder relies on.

tools: Stop using gtk_widget_show/hide

constraint editor: Stop using gtk_widget_show/hide

icon browser: Stop using gtk_widget_show/hide

gtk-demo: Stop using gtk_widget_show/hide

widget-factory: Stop using gtk_widget_show/hide

gtk: Stop using gtk_widget_show/hide

gtk_widget_set_visible and gtk_window_present
are better alternatives, and calling gtk_widget_show
on newly created widgets is no longer necessary
anyway.

Merge remote-tracking branch 'origin' into ccook/spelling-fixes

Merge branch 'wip/corey/tree-expander' into 'main'

treeexpander: Add hide-expander and indent-for-depth properties

Closes #4969

See merge request GNOME/gtk!5268

imcontextwayland: Set up OSK activation gesture on non text widgets

With GtkText and GtkTextView (and in extension, all their subclasses)
handling OSK activation activation, this gesture is only useful for
all text input widgets that are not subclasses of these 2 widgets,
e.g. the VTEs and crosswords of the world.

These still do need a hand in handling OSK activation, so only
set up the gesture for such cases.

gtktextview: Invoke OSK on button/touch taps that move/undo selection

If the ::release handler is invoked, the press/release happened without
drags in between. Additionally check that there is no selection at all.

This makes OSK invoked on taps that move the caret around, while tapping
in the selection invokes edition popup and text handles without bringing
in the OSK.

gtktextview: Claim drag gesture after an actual selection change

This way, the drag gesture lets the click gesture ::release handler
happen if there was no actual changes to the selected text (i.e.
too short drags). This matches the ::release handler behavior match
the situations in which the OSK was being invoked by the wayland
GtkIMContext.

gtktext: Invoke OSK on button/touch taps that move/undo selection

If the ::release handler is invoked, the press/release happened without
drags in between. Additionally check that the press did not happen within
the selection, and that there is no selection at all.

This makes OSK invoked on taps that move the caret around, while tapping
in the selection invokes edition popup and text handles without bringing
in the OSK.

gtktext: Claim drag gesture after an actual selection change

This way, the drag gesture lets the click gesture ::release handler
happen if there was no actual changes to the selected text (i.e.
too short drags). This matches the ::release handler behavior match
the situations in which the OSK was being invoked by the wayland
GtkIMContext.

imcontextwayland: Implement activate_osk() vmethod

This method does explicitly what the builtin GtkClickGesture in
the IM context is doing implicitly. The gesture will be dropped
after porting is done.

immulticontext: Implement activate_osk() vmethod

The multicontext lets this call go through to the active
underlying GtkIMContext.

imcontext: Add private activate_osk() vmethod

This method is so far private for both external GtkIMContext
implementations and external GtkIMContext users, and is meant
to activate the OSK in the environments where this may happen.

Revert "gtktext: Claim gesture more selectively"

This reverts commit ae54374a514aa33b34f95eee94b4a2848b8a16d9.

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

g-i: Add missing since annotations

See merge request GNOME/gtk!5271

g-i: Add missing since annotations

treeexpander: Minor formatting changes