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

Link the file mentioned as a reference for key values

Update Hungarian translation

Update Croatian translation

Merge branch 'make-pot-update' into 'main'

Add new printf patterns to make-pot

See merge request GNOME/gtk!5197

Add new printf patterns to make-pot

This script is used to extract our strings for translators
on damn lines, and passing these flags to xgettext makes
it put a hint into the pot file about strings that are
used as printf format strings.

Merge branch 'gdk-win32-settings-rework' into 'main'

GdkWin32 settings rework

See merge request GNOME/gtk!5193

GdkWin32: Add more GDK_NOTE calls for settings

GdkWin32: Set the "gtk-hint-font-metrics" property

Helps for https://gitlab.gnome.org/GNOME/gtk/-/issues/5257

GdkWin32: Set the "gtk-cursor-blink" setting

GdkWin32: Set the "gtk-cursor-theme-size" setting

Note that we have to set the logical value, that's why
we use GetSystemMetrics and not GetSystemMetricsForDpi.

GdkWin32: Detect the high contrast setting

And set the theme name accordingly ("Default-hc")

GdkWin32: Fix leak of screen HDC

GdkWin32: Remove special-casing for old Pango versions

We require Pango 1.50 now

GdkWin32: Remove old code

GdkWin32: Sort settings alphabetically

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

8455b9ac74 seems to have introduced a problem where we can wind
up focusing no widget at all if the `while (parent)` loop doesn't
find a widget it can successfully move the focus to. This 'fixes'
that by falling back to doing the previous thing if we make it
all the way through that loop without moving the focus. Thanks to
@coreyberla for a hint to improve the implementation.

Signed-off-by: Adam Williamson <awilliam@redhat.com>

gdk/wayland: Fix button mask calculation on button events

There's 2 things broken here:
- The mask was calculated on top of the GDK button (i.e. skipping
  4-7 buttons), so GDK_BUTTON4_MASK and GDK_BUTTON5_MASK were not
  assigned. This is now calculated on the (continuous) BTN_ evcodes
  so it is guaranteed that the next 2 physical buttons (i.e.
  back/forward) get these two places in the mask assigned.
- Furthermore, these buttons would be pushed to places in the
  modifier mask that they didn't belong to. It is now checked hard
  that only the first 5 buttons enable a modifier flag.

Overall, this ensures that no event masks with bonkers values are
forwarded, and that no stale implicit grabs are left after additional
buttons are pressed.

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

Update Turkish translation

Update Dutch translation

(cherry picked from commit 381c05607060c584779f9e6fab6f662ddfada1fa)

X11: Provide settings in logical pixels

This makes GtkSettings values on X11 match what we get on
other backends.

Reporting size settings in logical pixels (i.e for scale
== 1) is useful for properly supporting mixed-DPI setups.
As X11 doesn't support mixed-DPI setups anyway, XSettings
doesn't bother providing logical values. Thus we scale
from physical to logical values ourselves.

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

Post-release version bump

4.9.1

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

Revert "modules: Fix build on Visual Studio"

See merge request GNOME/gtk!5187

gdk: Bump the jpeg malloc limit

Bump the limit for memory use during jpeg loading
to 1GB, matching what gdk-pixbuf has for this.

Revert "modules: Fix build on Visual Studio"

This reverts commit acd9c12667a021936b30e0fa8f43d3a8c7133cf7.

This commit breaks the build with GLib main on all platforms,
and defining _GLIB_EXTERN arguably invades the GLib namespace.

A different fix for msvc will have to be found.

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

Be consistent in preconditions for async API

See merge request GNOME/gtk!5186

Be consistent in preconditions for async API

Do the same checks in all our finish functions.

Update Russian translation

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

Ebassi/since for main

See merge request GNOME/gtk!5185

docs: Add missing Deprecated annotations

docs: Add missing Since annotations

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

alertdialog: Small refactoring

See merge request GNOME/gtk!5184

Add a test for POTFILES.in

Add a test that runs make-pot. This will only pass
if you've updated po/POTFILES.in and .skip after
moving source files around.

Unfortunately, it won't catch new source files that
are missing.