contentserializer: Serialize to file:// if possible

Try to get a native file:// URI instead of any other GVFS
scheme, for interoperability with apps only handling file:// URIs.

This is what GTK3 Nautilus and Thunar do, so apps should be tuned
for this behavior.

See also https://bugzilla.xfce.org/show_bug.cgi?id=13845

Fixes: #5422

Merge branch 'work/zzag/wl-output-cleanup' into 'main'

gdk/wayland: Destroy zxdg_output_v1 upon GdkWaylandMonitor destruction

See merge request GNOME/gtk!5242

gdk/wayland: Clear GdkWaylandMonitor::output using g_clear_pointer()

Use g_clear_pointer() to make tear down code consistent.

gdk/wayland: Destroy zxdg_output_v1 upon GdkWaylandMonitor destruction

It prevents potentially leaking memory.

Merge branch 'file-filter-properties' into 'main'

filefilter: Add properties for the criteria

See merge request GNOME/gtk!5241

filefilter: Add properties for the criteria

Add construct-only properties for patterns,
suffixes and mime-types. This provides an
alternative way to specify these in ui files.

Related: #5350

Merge branch 'color-picker-portal' into 'main'

colorpicker: Always try the portal first

See merge request GNOME/gtk!5240

colorpicker: Always try the portal first

It does not hurt., and going forward, we want to
prefer portals.

Merge branch 'bilelmoussaoui/g-i' into 'main'

g-i: add missing array annotation to AlertDialog.set_buttons

See merge request GNOME/gtk!5234

Merge branch 'fix-upside-down-labels' into 'main'

gsk: Avoid a crash with negative scales

See merge request GNOME/gtk!5236

Merge branch 'drop-buildable-get-id' into 'main'

builder: Stop using GtkBuildable.get_id

See merge request GNOME/gtk!5239

Merge branch 'stringlist-prop' into 'main'

stringlist: Add a construct-only strings property

See merge request GNOME/gtk!5238

Drop gtk_widget_buildable_parser_finished

No need to have this no-op implementation.
This was the only implementation of this vfunc
in GTK.

builder: Stop using GtkBuildable.get_id

Drop the only implementation of this vfunc,
and stop using it. It isn't needed, and the
only implementation did the same thing GtkBuilder
does anyway.

stringlist: Add a construct-only strings property

This enables creating string lists in ui files
without using custom markup.

Related: #5350

Merge branch 'center-box-props' into 'main'

centerbox: Add properties for child widgets

See merge request GNOME/gtk!5237

centerbox: Add properties for child widgets

This provides and alternative to add children
in ui files.

Related: #5350

gsk: Avoid a crash with negative scales

Don't crash when both scale_x and scale_y are -1.

A test is included.

g-i: add missing array annotation to AlertDialog.(s|g)et_buttons

Merge branch 'builder-parse-attributes' into 'main'

builder: Parse PangoAttrList values

See merge request GNOME/gtk!5228

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

gtktext: Claim gesture more selectively

Closes #5351

See merge request GNOME/gtk!5230

Merge branch 'kjellahl/menubar' into 'main'

gtk_popover_menu_bar_set_menu_model(): Accept model == NULL

See merge request GNOME/gtk!5232

Merge branch 'shorter-im-context-ids' into 'main'

immmodule: Warn if GTK_IM_MODULE is bad

See merge request GNOME/gtk!5229

Merge branch 'wayland-foreign-v2' into 'main'

wayland: Also support the v2 of the importer

See merge request GNOME/gtk!5231

wayland: Also support the v2 of the importer

Commit 0ba8e16e1417fdb added support for the export
part of zxdg_foreign_v2, but neglected that the importer
has a v2 as well. Support it.

gtkpopovermenubar: Make model nullable

The description of gtk_popover_menu_bar_new_from_model() and
gtk_popover_menu_bar_set_menu_model() says that model is nullable.
Make it so.

gdksurface: Do not consider GDK_TOUCH_END/CANCEL as popup-dismiss worthy

GDK_TOUCH_END deserves the same treatment than GDK_BUTTON_RELEASE, since it's
subject to the same circumstances (popping up a menu on long press would be
immediately dismissed on release if we handled them there). Ideally, we would
want to match releases that we obtained a press for while grabbed, but as
the popup is also dismissed on GDK_BUTTON_PRESS/GDK_TOUCH_BEGIN, there's no
use for this tracking.

And GDK_TOUCH_CANCEL sounds weird as a reason to dismiss popups, just like
crossing events would.

Related: https://gitlab.gnome.org/GNOME/mutter/-/issues/2512

gtktext: Claim gesture more selectively

Even though button 1 (or touch down) presses do most often have
an effect in one way or another (starting drag, moving focus,
starting selection, ...), there is one situation that they do
immediately nothing: When clicking on the entry does not move
the text caret around. Dragging might start a selection, but
the entry did not do anything just yet, and an immediate
button/touch release should remain at "did nothing".

And that is precisely the hint that the Wayland IM context's click
gesture takes, clicks that do not scroll nor move the caret around,
having the GtkText not claim the gesture in that situation makes
the IM gesture able to do its thing without in-fighting.

This is typically not a problem when the GtkText is embedded in
another GtkEditable implementation (e.g. GtkEntry), since the
IM gesture is inactive and capturing from the parent widget, so
gets a pass that it otherwise doesn't get when both gestures are
in the same widget. This makes it work regardless of GtkText not
being a child of a composite widget, like NautilusQueryEditor
and AdwRowEntry.

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

Merge branch 'wip/fix-tiling-without-gtk-shell' into 'main'

wayland/surface: Set the tiling state from xdg-shell too

See merge request GNOME/gtk!5227

immodule: Accept 'simple' and 'none'

Everybody (including myself) gets this wrong,
so accept 'simple' and 'none' as shorthands
for the official IDs 'gtk-im-context-simple'
and 'gtk-im-context-none'.

immmodule: Warn if GTK_IM_MODULE is bad

If we don't find an immodule matching the
ids in the GTK_IM_MODULE envvar, print a
warning.

builder: Parse PangoAttrList values

Allow specifying PangoAttrList properties in the
serialized format supported by pango_attr_list_to_string().

Test included.

wayland/surface: Set the tiling state from xdg-shell too

We plumbed it from the gtk-shell configure event, but didn't for the
equivalent state passed via xdg_toplevel.configure.

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

gsk: Add an assertion to help static analysis

See merge request GNOME/gtk!5224

stringsorter: Fix one case of collation handling

When not ignoring case, and not collating, we were
returning sort keys that were already freed. Oops.

Pointed out by clang.

gdk: Fix possible memory errors

clang complained that we may end up jumping
to the cleanup code without initializing data
in the jpeg code. Always initialize data to
NULL to prevent that eventuality.

composetable: Fix a memory leak

Pointed out by clang.

gsk: Add an assertion to help static analysis

clang gets wild ideas about negative radii otherwise.

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

Fix spelling errors

See merge request GNOME/gtk!5223

Drop redundant glib version checks

We require GLib 2.72 now.

gdk: Clean up gdkprivate.h

Move apis to more specific headers.

gdk: Rename GDK_SURFACE_TEMP

We use this value only for drag surfaces, so call it that.

Fix spelling errors

These were pointed out by codespell.

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

file dialog: Use portals

See merge request GNOME/gtk!5221

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

gtk-demo: Add some keywords

See merge request GNOME/gtk!5220

filedialog: Use portals

We want this new API to be portal-first.
Make it so.

gtk-demo: Add folders to the clipboard demo

This is handy for testing a case where folder
dnd does not work with the file transfer portal.

See #5348

gtk-demo: Add some keywords

Make sure that dnd and drag-and-drop yield the right
(and the same) set of demos.

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

doc fix: various spelling mistakes corrected

See merge request GNOME/gtk!5215

Merge branch 'win32-hotspot' into 'main'

gdk/win32: fix cursor hotspot

See merge request GNOME/gtk!5216

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

doc/spelling changes: gtkselectionmodel.c, 'call this when the* selection changes'

Merge branch 'radioactiveman-main-patch-52462' into 'main'

label: Fix markup doc example

See merge request GNOME/gtk!5201

label: Fix markup doc example

Merge branch 'ebassi/list-doc-links' into 'main'

docs: Add links to the list widgets table

See merge request GNOME/gtk!5210

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

Inspector: Add a missing event type

See merge request GNOME/gtk!5218

Merge branch 'filechooser-set-choice-fix' into 'main'

filechooser: Fix gtk_file_chooser_set_choice again

Closes #5346

See merge request GNOME/gtk!5219

filechooser: Fix gtk_file_chooser_set_choice again

If the choice was in the form of a GtkDropDown, the code failed to
find the widget and so the choice never got set. Fixes #5346.

gdk/win32: fix cursor hotspot

This seems to be a regression from commit c06b1cc103b8 "Win32: Re-work cursor handling".

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

Alertdialog: Cosmetics

events: Drop an unused variable

Inspector: Add a missing event type

We did not have a name for the new touchpad hold
events.

To prevent this from happening again in the future,
add a static assertion.

Documentation fix: Various spelling mistakes (and one use of the wrong verb) across a few domains that show up in documentation.

| domain | current | suggestion |
|--------|---------|------------|
| [GtkButton](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkbutton.c#L691)       | "Buttons can has a flat appearance" | "Buttons can have a flat appearance" |
| [GtkCenterBox](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkcenterbox.c#L275) | "pas %NULL" | "pass %NULL" |
| [GtkEditable](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkeditable.c#L466)   | "the text to append" | "the text to insert" |
| [GtkFlowbox](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkflowbox.c#L4921)    | "the are equal" | "they are equal" |
| [GtkSelectionModel](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkselectionmodel.c#L550) | "Call this when a the selection changes" | "Call this when a selection changes" |
| [GtkWIndow](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkwindow.c#L5321)                | "(e.g. the user or the window manager" | "(e.g. the user or the window manager)" |

docs: Add more links to the list widgets description

docs: Add links to the list widgets table

Make it easier for readers to go to the appropriate replacement for the
old GtkTreeView-and-friends API.

Merge branch 'focus-no-parent' into 'main'

focus: fall back to old behaviour if we didn't focus anything

See merge request GNOME/gtk!5189

Merge branch 'docs-update' into 'main'

docs: Update Common Questions section on list views

See merge request GNOME/gtk!5209

Fix typo in 4 to 5 migration page

Update Common Questions section on tree views

Point users to GtkListView instead of the
deprecated GtkTreeView.

Merge branch 'gdkwaylandtoplevel' into 'main'

gdk/wayland/surface: Move more members to `GdkWaylandPopup` and `GdkWaylandToplevel`

See merge request GNOME/gtk!4227

Merge branch 'win32-inhibit' into 'main'

gdk-win32: implement basic inhibit-system-shortcuts

See merge request GNOME/gtk!5206

Merge branch 'export-handle-refactor' into 'main'

Refactor surface handle export

See merge request GNOME/gtk!5205

gdk-win32: implement basic inhibit-system-shortcuts

This is largely adapted from commit 83027c68f112 ("11: Implement
inhibit_system_shortcuts API"), with similar rationale:

    To implement the inhibit_system_shortcuts API on X11, we emulate the
    same behavior using grabs on the keyboard.

    To avoid keeping active grabs on the keyboard that would affect
    other X11 applications even when the surface isn't focused, the X11
    implementation takes care of releasing the grabs as soon as the
    toplevel loses focus.

Note that Windows has low-level keyboard hooks that could help achieve
the expected behaviour. This is implemented by spice-gtk & gtk-vnc for
example, but correctness isn't obvious. I left a TODO comment.

This patch helps implementing remote desktop widgets with GTK4, since
currently on win32 backend Alt-Tab and such are always left to the
system unless there is keyboard grab (which can't be requested by the
client API anymore, afaict).

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

wayland: Reimplement export_handle

Reimplement the backend API on top of the new async
calls.

window: fixups

wayland: fixups

wayland: Support the v2 protocol in both places

The old api should better support both versions of the
foreign protocol too.

wayland: Add zxdg_foreign_v2 protocol support

window: Use gdk_toplevel_export_handle

Add gdk_toplevel_export_handle

This code is backend-specific, and should not live
in gtkwindow.c.

gdk-win32: fix unused variable warnings

../gdk/win32/gdksurface-win32.c: In function 'gdk_win32_get_window_size_and_position_from_client_rect':
../gdk/win32/gdksurface-win32.c:3753:20: warning: unused variable 'impl' [-Wunused-variable]
 3753 |   GdkWin32Surface *impl = GDK_WIN32_SURFACE (window);
      |                    ^~~~
../gdk/win32/gdksurface-win32.c: In function 'gdk_win32_surface_get_queued_window_rect':
../gdk/win32/gdksurface-win32.c:5129:20: warning: unused variable 'impl' [-Wunused-variable]
 5129 |   GdkWin32Surface *impl = GDK_WIN32_SURFACE (surface);

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

Merge branch 'wayland_display_geometry' into 'main'

wayland: calculate union of geometry of all monitors for toplevel bounds

See merge request GNOME/gtk!4790

Merge branch 'cursor-size' into 'main'

GtkSettings: Provide settings in logical pixels also on X11

Closes #5223 and #5230

See merge request GNOME/gtk!5172

Merge branch 'wip/carlosg/back-forward-mask' into 'main'

gdk/wayland: Fix button mask calculation on button events

Closes #5301

See merge request GNOME/gtk!5191

Merge branch 'add-link-to-docs' into 'main'

Link the file mentioned as a reference for key values

See merge request GNOME/gtk!5198

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

docs: Add deprecation notices for properties

See merge request GNOME/gtk!5204

Merge branch 'ccook/gtk_aspect_frame_doc_fix' into 'main'

doc/comment fix: @self misspelled as self@

See merge request GNOME/gtk!5200

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

Add new printf patterns to make-pot

See merge request GNOME/gtk!5188

docs: Add deprecation notices for properties

Merge branch 'fix-color-dialog-button-new' into 'main'

colordialogbutton: Actually accept NULL

Closes #5321

See merge request GNOME/gtk!5202

gtk-demo: Improve language handlng

In the font features demo, don't add a
item for the default language to the dropdown,
that does not make sense.

gtk-demo: Add a few mssing language names

gtk4-demo: Tweak the pickers demo

Redo the file picker a bit.

po: Match make-pot

We are not normally using the gtk40-pot target to
generate the gtk40.pot file. On the off chance that
somebody does, lets make sure we pass the same
arguments to xgettext here as in the make-pot script
that is used on damn lies.

Add a test for picker button constructors

This turns out to be hard enough that a test
is useful.

colordialogbutton: Actually accept NULL

As pointed out in #5321, my previous fix was not
good enough.

Fixes: #5321

wayland: calculate union of geometry of all monitors for toplevel bounds

Signed-off-by: Colin Kinloch <colin.kinloch@collabora.com>

doc/comment fix: the ref to @self was misspelled as self@ for gtkaspectframe.h/gtk_aspect_frame_get_child

Noticed this while reviewing the gi-docgen docs for GtkAspectFrame while developing some java bindings.
It's my understanding that @self was intended; as it would cause gi-docgen to interpret it as a reference to
the the GtkAspectFrame pointer named 'self'.

Merge branch 'fix-color-button-new' into 'main'

color/fontdialogbutton: Fix constructors

Closes #5321

See merge request GNOME/gtk!5199

color/fontdialogbutton: Fix constructors

The arguments were meant to be nullable.
Make it so!

Fixes: #5321