istring: fix istring_prepend() on malloc transition

When transitioning from internal to malloc, the strings were placed in
the wrong order to g_strconcat(). This fixes an issue with undo where
if you hit the boundary in just the right way, your undo stack will do
unexpected things.

Fixes #5506

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

Add some more text undo tests

See merge request GNOME/gtk!5410

Update Russian translation

Add some more text undo tests

Unsuccessfully trying to reproduce #5506.

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

Deprecate gdk_display_put_event

See merge request GNOME/gtk!5409

Deprecate gdk_display_put_event

The documentation already tells you not to
use this function.

Ifdef the remaining users in the X11 backend.

Merge branch 'macos-stable-dnd' into 'main'

macos: Drag and drop should not freeze applications

See merge request GNOME/gtk!5291

Merge branch 'wip/corey/popover-accel' into 'main'

gtkmodelbutton: Ensure that accel label is always aligned to end

Closes #5504

See merge request GNOME/gtk!5407

gtkmodelbutton: Ensure that accel label is always aligned to end

When the model button just has a text label and accel text,
the button fills and the accel label is implicitly aigned to end.
When there's also a icon, even though it's not shown (because
icons are only shown if there's no text), the button doesn't fill
and the accel ends up not aligned (assuming one of the other buttons
is longer). Ensure that the accel label is aligned to the end.

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

Merge branch 'a11y_check_null_path' into 'main'

a11y: Avoid crashing if GtkAtSpiContext::path is NULL

See merge request GNOME/gtk!5405

Merge branch 'doc-improvements' into 'main'

Two docs improvements

See merge request GNOME/gtk!5406

docs: Improve gtk_window_present description

Move most of the documentation from gtk_window_present_with_time and
also reword it a bit to make it more understandable.

docs: Don't mention deprecated API in gtk_widget_set_visible

That statement is true, but since that API is deprecated, it doesn't
need to be there anymore to avoid confusing people.

a11y: Use fallback logic when GApplication has no DBus object path

Use the fallback logic to generate the base path for the GtkAtSpiRoot
if the GApplication has no DBus object path to guarantee that
the base path will not stay NULL.

a11y: Check if path is NULL when removing from cache

`gtk_at_spi_cache_add_context()` checks if the GtkAtSpiContext's path
is NULL before inserting the context object into the hash table.
Do the same in `gtk_at_spi_cache_remove_context()` to avoid a NULL
pointer dereference in `g_str_hash()` during the hash table lookup
if a context with NULL path is removed. That can happen when the
GtkAtSpiRoot::base_path is NULL, which, in turn, can happen if
`g_application_get_dbus_object_path()` returns NULL.

  ==394047==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x7fd1966f8b84 bp 0x7fff11e3ded0 sp 0x7fff11e3de58 T0)
  ==394047==The signal is caused by a READ memory access.
  ==394047==Hint: address points to the zero page.
      #0 0x7fd1966f8b84 in g_str_hash (/usr/lib/libglib-2.0.so.0+0x37b84)
      #1 0x7fd1966f9c09 in g_hash_table_contains (/usr/lib/libglib-2.0.so.0+0x38c09)
      #2 0x7fd196062c10 in gtk_at_spi_cache_remove_context ../gtk/a11y/gtkatspicache.c:447
      #3 0x7fd19606e0a9 in gtk_at_spi_root_unregister ../gtk/a11y/gtkatspiroot.c:653
      #4 0x7fd196067f58 in gtk_at_spi_context_unrealize ../gtk/a11y/gtkatspicontext.c:1559
      #5 0x7fd195ced97f in gtk_at_context_unrealize ../gtk/gtkatcontext.c:668
      #6 0x7fd195f5576e in gtk_widget_unroot_at_context ../gtk/gtkwidget.c:2399
      #7 0x7fd195f55bd2 in gtk_widget_unroot ../gtk/gtkwidget.c:2499
      ...

Fix code style

gdk: Update docs

Update the docs for gdk_display_get_monitor_at_surface.

Merge branch 'gdk-display-get-monitor-is-nullable' into 'main'

gdkdisplay: Document get_monitor_at_surface can return NULL

Closes #5075

See merge request GNOME/gtk!4917

Merge branch 'more-filesystemmodel-cleanups' into 'main'

filesystemmodel: Crop tree model cruft

See merge request GNOME/gtk!5402

Merge branch 'win-file-ext' into 'main'

filechoosernativewin32: Set default extension

See merge request GNOME/gtk!5266

filechoosernativewin32: Set default extension

filesystemmodel: Drop tree model cruft

We no longer need the placeholder item for the
editable, so drop it.

filesystemmodel: Rename private header

Merge branch 'filesystemmodel-cleanups' into 'main'

filesystemmodel: Cleanup unused defines

See merge request GNOME/gtk!5401

gtkfilesystemmodel: Use g_clear_* helpers

Saves some code.

filesystemmodel: Use G_DECLARE_FINAL_TYPE()

Clean up boilerplate. It's a private object anyway.

filesystemmodel: Cosmetics

Adjust code style, and shuffle some functions around. Remove
unhelpful doc comments.

filesystemmodel: Use g_set_object

filesystemmodel: Make function private to gtkfilesystemmodel.c

The _gtk_file_system_model_update_file() function is not used outside
GtkFileSystemModel, so no need to expose it in the header.

Shuffle it around in code, and remove it from the header.

filesystemmodel: Cleanup unused defines

They were made unused with the removal of GtkTreeModel interface
implementation.

Merge branch 'werror-fixes' into 'main'

gtk-launch: Remove compile-time warning

See merge request GNOME/gtk!5400

Merge branch 'wip/corey/calendar' into 'main'

calendar: Fix marked days

See merge request GNOME/gtk!5399

calendar: Set marked days immediately

The marked days are set only as part of gtk_calendar_select_day().
This is insufficient, especially because the day-selected signal
is emitted after the marked days are set in gtk_calendar_select_day().

calendar: Only apply the marked_day to the current month

The marked day gets applied to the current month, and either the previous
or subsequent month if they are visible within the current month.
This doesn't make any sense and likely was an accidental regression
in a6f9052cf1c1c7b4224834ed18a63e70477d8729.  Clarify the docs.

calendar: Add css style for marked days

As part of a6f9052cf1c1c7b4224834ed18a63e70477d8729, marked days lost
their style, essentially making that function worthless.  Previously,
they were simply bolded, but that doesn't give them proper justice.

Merge branch 'wip/phako/fix-activate-action-annotation' into 'main'

widget: Mark format as nullable in activate_action

See merge request GNOME/gtk!4653

Merge branch 'main' into 'main'

Fix a typo

See merge request GNOME/gtk!5170

build: Turn off tautological-constant-out-of-range-compare warning

This issues a warning when an enum value is compared to a value that is
out of range for the enum.

We do this a lot, either when using -1 for undefined values or when
comparing array sizes to enum values like so:

  enum {
    ONE,
    TWO,
    THREE
  } some_enum_value;
  const char *names= { "one", "two", "three" };

  g_assert (some_enum_value < G_N_ELEMENTS (names));

gtk-launch: Fix unused variable warning on macOS

ci: Disable -Werror on all subprojects

Would be nice if meson had a nicer way of doing this, but...

glcontext: Avoid gcc warning on non-egl build

gtk-launch: Remove compile-time warning

This warning causes a -Werror build to fail.

Merge branch 'wayland-surface-split3' into 'main'

Split gdksurface-wayland.c

See merge request GNOME/gtk!5217

Drop gdkpopup-wayland-private.h

It is empty now.

waylandsurface: Introduce a few vfuncs

waylandpopup: Drop unused exports

waylandtoplevel: Drop unused exports

More cleanups

Misc cleanups

Detangle compute_size, among other things.

wip: Split off GdkWaylandPopup

Like the GdkWaylandToplevel split-off, this needs some
more cleanup.

wip: Split off GdkWaylandToplevel

This still needs some more cleanup.

wayland: Rearrange the surface code

wayland: Split public headers

Split gdkwaylandsurface.h into itself,
gdkwaylandtoplevel.h and gdkwaylandpopup.h.
This is in preparation for splitting the
types for real.

wayland: Split display_server sub-structs

Move these to GdkWaylandToplevel and GdkWaylandPopup
as appropriate.

wayland: Move next_layout into popup and toplevel

This is where it belongs.

wayland: Move pending toplevel state to GdkWaylandToplevel

Another toplevel-only thing.

wayland: Move input_grab_seat to GdkWaylandPopup

This is popup-only functionality.

wayland: Move reposition_token to GdkWaylandPopup

This is only used for popups, so move it there.

wayland: Move shortcuts_inhibitors to GdkWaylandToplevel

Another toplevel-only feature.

wayland: Move imported_transient_for to GdkWaylandToplevel

Another toplevel-only thing.

wayland: Move geometry hints to GdkWaylandToplevel

Another toplevel-only thing.

wayland: Move title to GdkWaylandToplevel

This is a toplevel property.

wayland: Move drag surfaces to their own file

wayland: Drop create_dnd_surface

It is only used in one place.

Merge branch 'swap-3-4' into 'main'

docs: Place example-{3,4} in numerical order

See merge request GNOME/gtk!5319

Merge branch 'gtk4-fix-xdg-activation-set-surface' into 'main'

gdk/wayland: set requesting surface to xdg-activation

See merge request GNOME/gtk!5293

Merge branch 'drop-file-dialog-shortcuts' into 'main'

filedialog: Drop shortcut folders

See merge request GNOME/gtk!5396

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

filedialog: Never set the chooser's file

Closes #5496

See merge request GNOME/gtk!5393

Merge branch 'macos-command-symbol' into 'main'

macos: Use Command key symbol in Keyboard shortcuts window

See merge request GNOME/gtk!5296

filedialog: Drop shortcut folders

The filechooser portal does not support this,
and we are defaulting to using the portal now.
Lets not offer a non-functional API.

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

testfiledialog: Be more lenient

See merge request GNOME/gtk!5395

testfiledialog: Be more lenient

Accept both open_multiple and open-multiple.

macos: Command should not come from gettext

Instead, just provide the right character.

See also
https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkaccelgroup.c#L916.

Merge branch 'test-file-dialog' into 'main'

Test file dialog

See merge request GNOME/gtk!5394

Add a test for GtkFileDialog

filedialog: Cosmetics

Merge branch 'wip/corey/dnd-use-cursor' into 'main'

Fix accidental DnD

See merge request GNOME/gtk!5275

filedialog: Never set the chooser's file

Instead, get by with setting the name and hope the filechooser figures
things out by itself.

Fixes #5496

Merge branch 'openuri-activation-token' into 'main'

Pass an activation token to OpenURI

See merge request GNOME/gtk!5389

Merge branch 'filechooser-multi-folder' into 'main'

filechooser: Don't assert for possible cases

Closes #5494

See merge request GNOME/gtk!5392

filechooser: Don't assert for possible cases

switch_to_selected_folder can be called when the
selection contains more than one item. Handle it
like it used to be handled: switch to the first
folder we find.

Fixes: #5494

Pass an activation token to OpenURI

The OpenURI portal can take an activation_token
nowadays. So it give one.

x11: Make a call NULL-safe

Make it safe to call gdk_x11_app_launch_context_get_startup_notify_id
with a NULL info.

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

Drop gtkmain.c from POTFILES

See merge request GNOME/gtk!5390

Drop gtkmain.c from POTFILES

No longer any translations here.

Merge branch 'deprecate-toggled' into 'main'

Deprecate gtk_toggle_button_toggled

See merge request GNOME/gtk!5388

Deprecate gtk_toggle_button_toggled

The function is documented as useless.

Update Swedish translation

Merge branch 'wip/carlosg/startup-vs-focus' into 'main'

Consider startup vs focus requests for the Wayland platform

See merge request GNOME/gtk!5386

gdkdisplay: Deprecate gdk_display_notify_startup_complete()

We have various layers where we store the startup ID for a request,
since this API does not have a GdkToplevel that we can refer about
for the Wayland platform, this is the most obvious candidate to
start untangling these various layers.

Deprecate this call, it is already unused in the gtk/ side.

gtkapplication: Do not call gdk_display_notify_startup_complete()

This should do nothing worthwhile anymore, the X11/Wayland GtkApplication
implementations do already pass the startup ID from the platform_data
via windowing specific APIs, and the application handling the request
via show()/present() should trigger the activation request.

gtkwindow: Shuffle gdk_toplevel_set_startup_id() calls

While this used to be tangential to windows showing or requesting
focus, the xdg-activation Wayland protocol does merge both concepts
together.

But also, for a correct interaction with the compositor, the
toolkit should ideally merge the activation request resulting from
both into the same one, so that the gdk_toplevel_focus() request
replies to the startup token that started the application and
correct focus-stealing prevention/etc happens, instead making up
one just in time for the focus request.

This kind of requires doing things in the right order, a show()
request on the GtkWindow should activate any pending activation
token on the toplevel, a present() request should additionally
create a new token if there was none pending. And
xdg_activation_v1_activate() should happen once on both.

Shuffle the gdk_toplevel_set_startup_id() calls so that this
happens in the right order for Wayland, while making X11 happy
too.

gdk/x11: Emit "remove" s-n message from gdk_toplevel_set_startup_id

In the way towards deprecating gdk_display_notify_startup_complete(),
make gdk_toplevel_set_startup_id() on X11 perform this piece of messaging
itself. It should be harmless that the message is emitted twice, if
callers do still use that API.

gdk/wayland: Perform xdg_activation on gdk_toplevel_set_startup_id

This call has everything to perform activation as specified by the
xdg_activation protocol, notably a surface to activate as opposed to
gdk_display_notify_startup_complete().

Make activation happen here, so that the surface gets activated when
its gets a startup ID assigned.

gtkwindow: Minor refactor

Move the handling of the startup ID to a separate function, since
this will be called from several places.

Merge branch 'fix-introspection' into 'main'

meson.build: build introspection according to option only

See merge request GNOME/gtk!5382

meson.build: build introspection according to option only

The way the check is written, if the build is native, then the
introspection option has no effect.

Particularly yocto project does want to disable introspection in
native builds and enable it in cross builds (both via the option),
and without this patch the former is not possible.

Signed-off-by: Alexander Kanavin <alex@linutronix.de>

Merge branch 'nullable-gdk' into 'main'

gdk_display_open: Mark arg as nullable

See merge request GNOME/gtk!5380

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

ci: Enable introspection for the docs build

See merge request GNOME/gtk!5383

Merge branch 'direction-from-language' into 'main'

Determine direction from language

See merge request GNOME/gtk!5385