Add tweaks

Allow setting the puzzle size, and allow chosing between
the rose and the radioactive animation. Change the default
size to 3. 6 is just too hard.

puzzle: Add a reshuffle button

Keep aspect ratio of the image

Since we don't have a good way to control the window size tightly,
make the content keep aspect ratio.

demo: Add the sliding puzzle demo

Merge branch 'master' into 'master'

Fix reference to user styles in gtkstyleprovider.h

See merge request GNOME/gtk!184

font-button: Suggest non deprecated function in docs

gtk_font_button_get_font_name() has been deprecated in 3.22 and
its public API is removed from GTK4.

Fix reference to user styles in gtkstyleprovider.h

Merge branch 'wip/move-to-rect-public' into 'master'

gdk: Make gdk_surface_move_to_rect public

See merge request GNOME/gtk!179

SizeRequest: Round px values up for min CSS sizes

Otherwise, requesting a min size in em where the equivalent in px had a
fractional part would lead to the widget getting allocated 1 too few px.
You could see this in the CSS property vs. allocation in the Inspector.

Note that margin/border/padding are left alone: the rationale is that we
do as browsers do, and Benjamin said we already do that for those,
whereas his tests on min-(width|height) showed otherwise. My subsequent
analysis indicated it to be far less clear-cut than that, but he remains
unconvinced that we should ceil() all the things! So just do these ones.

https://gitlab.gnome.org/GNOME/gtk/issues/1088

gdk: Make gdk_surface_move_to_rect public

This is the API used by GtkMenu to properly position menus on the screen
without requiring GTK to query the menu window's position or the work
area of where the window is positioned. It makes it possible to position
popup windows properly when using Wayland.

Make this API available to external users so custom popup windows can be
positioned properly as well.

Related: https://gitlab.gnome.org/GNOME/gtk/issues/997

popover: Add missing close paren in docs

colorbutton: Remove priv pointer

aspectframe: Remove unused member

bin: Remove public GtkBinPrivate typedef

statusbar: Remove garbage from header file

Nobody will ever read those comments; documentation belongs into the
soruce file so gtk-doc actually picks it up.

statusbar: Remove priv pointer

widget: Add docs for {get,set}_focus_child

With get_ being still private.

gl renderer: use w axis vector from graphene

Instead of initializing our own one every time.

Merge branch 'patch-1' into 'master'

GtkWindow: Handle non-square icons with height > width correctly

Closes #657

See merge request GNOME/gtk!170

GtkWindow: Handle non-square icons with height > width correctly

widget: Fix example code in doc

Merge branch 'gesture-docs' into 'master'

Gesture stuff

See merge request GNOME/gtk!169

x11: Don't set NET_WM_PID when sandboxed

It is not useful, and some window managers misinterpret it and
add some "runs as root" indication to the window decoration.

See https://github.com/mate-desktop/marco/issues/301

gdk: Add a private api to find sandboxes

This will be used in more places in the future.

events: Mark static function as such

gdk: Get rid of gdk_event_free()

Events are objects, so use g_object_unref().

gesturemultipress: Don’t fire ::released after ::cancel

Causing a grab in the handler for ::pressed by, e.g., popping up a
context menu will cause the gesture to be canceled and, subsequently,
::end and ::released to be fired, all while the button is still
physically pressed. That results in no event being available to the
::released handler and garbage coordinates, given that
gtk_gesture_get_point() returns FALSE.

Emitting ::released can be avoided by checking the return value
gtk_gesture_get_point().

gesture: Fix code snippet

GTK_EVENT_SEQUENCE_ACCEPTED seems to never have been a valid enumeration
value. GTK_EVENT_SEQUENCE_CLAIMED is the closest in meaning.

gesture: Fix get_last_event() docs

Querying the event sequence of a gesture will always yield NULL for
non-touch events, but passing NULL in to calls to
gtk_gesture_get_last_event() is a perfectly valid use case.

gdkseatdefault: Don't hide GdkSurface on grab failure

Application is not expecting that.

Bug found due gdk_seat_grab() failure on Lock Screen. When user
Unlock the screen, the application is visible but does not receive
enter-event any more on X11/GNOME.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1485968
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1571422

Signed-off-by: Victor Toso <victortoso@redhat.com>
Signed-off-by: Carlos Garnacho <carlosg@gnome.org>

gtkpopover: Fix thinko in key press event type check

That code branch is meant to check for key events, seems obvious we want
GDK_KEY_PRESS, not GDK_BUTTON_PRESS (which also broke the branch right
below).

Makes us all able to dismiss popovers again.

Revert "popover: Fix setting priv->button_pressed in ::event handler"

This reverts commit d638ff3afa7f3ccfebf2d5de23616109742d622b.

Doesn't look like the right fix. Another one coming that fixes dismissing
through both pointer and keyboard.

Remove an unused struct

Positive side-effect: visual c no longer has an empty struct
to complain about.

popover: Fix setting priv->button_pressed in ::event handler

There were two cases in that if/else cascade handling the
event_type == GDK_BUTTON_PRESS case, so priv->button_pressed never got
set.

gdkscreen-x11: Remove redundant typedef

Causes warnings with clang.

gl renderer: Remove an outdated comment

Merge branch 'master' into 'master'

icontheme: Keep dir_mtimes in order

Closes #1115

See merge request GNOME/gtk!161

filechooserwidget: Don't chain up in ::event

GtkWidgetClass.event is NULL.

gl renderer: Use offsets for more node types

Update Polish translation

a11y: Stop using ::event

We can just as well use notify::has-focus for the purpose of
focus tracking, and we can at the same time avoid emitting the
deprecated AtkObject::focus-event signal.

ci/msys2: force disable vulkan

A recent dependency change in MSYS2 made it pull in vulkan, which made
meson think it's available but it somehow links against the system vulkan dll
instead.

Disable vulkan for now.

SpinButton: Fix an obviously wrong arg description

:climb-rate is not about what you get when you single-click on a button,
as this implied: it's what happens if you hold down a button or a key.
Fix the description of @climb_rate to new(), and while here, mention the
key in the blurb of :climb-rate itself.

icontheme: Keep dir_mtimes in order

Don't reverse the order each time we insert a theme. Reverse it only
once, after all themes have been loaded.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1115

Merge branch 'kill-key-events' into 'master'

Kill key events

See merge request GNOME/gtk!160

Merge branch 'issue792' into 'master'

wayland: check native surface for crossing events

Closes #792

See merge request GNOME/gtk!148

Drop the ::key-press/release-event signals

These are the last event-type specific signals,
we are just relying on the generic ::event signal now.

treeview: Stop using ::key-press-event

Use a generic ::event handler instead.

spin button: Stop using ::key-release-event

Just use the generic ::event signal.

font chooser dialog: Stop using ::key-press-event

Use the generic ::event instead.

button: Stop using ::key-release-event

Use the generic ::event signal instead.

file chooser widget: Stop using ::key-press-event

Use a generic ::event handler instead.

file chooser entry: Stop using key-press-event

Just use the generic ::event signal for now.

entry: Stop using ::key-press-event

Use a generic ::event handler instead.

StyleContext: Don't advise using invalidate()

It doesn't exist anymore! Also, set_display() is not needed/used in the
demo cited, and we were still referring to gtk3-demo, not gtk4-demo.

Range: Up should only mean ++ if we are a GtkScale

The last round of patches to get the desired direction of value move in
response to scrolls/keypresses on scales had the inadvertent side effect
of giving the opposite direction on scrollbars. Seeing as gtkrange.c is
already a collection of hacks, add another so that fix only holds if the
instance is a GtkScale, since that is what those patches were aimed at.

Close https://gitlab.gnome.org/GNOME/gtk/issues/1065

emoji completion: Stop using key-press-event

The generic ::event signal will do.

Remove key-press-event from more docs

This signal will be going away, so stop mentioning it so much.

gtk-demo: Avoid key-press-event

popover: Drop key_press_event handler

We can do the same in the generic ::event handler.

emoji: Hide recent section when empty

This is part of https://gitlab.gnome.org/GNOME/gtk/merge_requests/143
by Julian Sperber.

widget: Properly remove event controllers in finalize

We were mutating the list while iterating over it. This was not a
problem before since remove_controller just set the controller pointer
to NULL instead of actually removing it from the list of controllers.

Add GtkEventControllerKey to the docs

widget: Remove controller sequence-state-changed signal connection

We can avoid a signal connection per event controller (and the
EventControllerData struct) since every event controller knows the
widget it's attached to.

Add basic searchbar set_key_capture_widget tests

Make sure we're correctly resetting the internal key capture widget
pointer when the widget goes away.

searchbar: Add missing GDK_AVAILABLE_IN_ALL marker

snapshot: Remove superfluous typedef

We're already doing this in gtktypes.h

Fixes #214

widget: Directly notify gestures of grab

Since each widget has a list of event controllers now, we don't need to
connect to ::grab-notify of the widget for every controller.

widget: Fix measure docs

Fix a typo and don't ever pass NULL to a ::measure() implementation.

gl renderer: Add helper for printing a render node tree

gestures demo: widget property is not writable

Use gtk_widget_add_controller instead.

scrollbar: Remove get_wheel_delta

Unused and undocumented.

cssvalue: Remove typechecks

This path is more than hot and those type checks ruin any kind of real
life profiling.

Update Friulian translation

emoji: Improve section scrolling

Leave some space above the section heading when scrolling.

This is a part of https://gitlab.gnome.org/GNOME/gtk/merge_requests/143
by Julian Sperber.

Merge branch 'wip/restart-cursor-animation' into 'master'

wayland: Fix restarting cursor animation

See merge request GNOME/gtk!149

Merge branch 'paste-scrolling' into 'master'

textview: Don't scroll for pastes in another view

See merge request GNOME/gtk!152

Merge branch 'pages-docs' into 'master'

Publish master docs with gitlab pages.

See merge request GNOME/gtk!135

gtkemojicompletion.c: Don't use g_autoptr()

This code is also built with non-GCC/non-CLang compilers, so don't use a
GCCism here.

gdkcairocontext*.h: Include cairo.h consistently

The other sources include cairo.h by just using #include <cairo.h>, so
do likewise here.

textview: Don't scroll for pastes in another view

GtkTextView scrolls to the insertion point when the text
buffer signals a paste is done. This is wrong when there
are multiple views on the same buffer, and the paste
happened in another view.

To fix this, flip the handling of the scroll_after_paste
boolean to only be TRUE if we know that we want to scroll.

Update Friulian translation

Update Friulian translation

appchooserdialog: Fix memory leak

appchooserdialog: Don't fire notify::heading twice

The gtk_app_chooser_dialog_set_heading() function do emit
notify::heading. Since the setter simply calls the function,
the setter itself shouldn't emit a notify signal by itself.

aboutdialog: Fix memory leak

video: Fix memory leak

examples: Remove unused variables

snapshot: Fix typo in documentation comment

label: Rearrange struct members

Rearranging so can save us 8 bytes per GtkLabel
on 32/64 bit aligned memory.
Not a big count, but we get it for free.

textview: Always show select-all button in touch popup

"Select all" action is possible regardless of the text selection state
or if the text is editable or not.

textview: Reset bubble source id once run

The bubble_timeout_id was reset only on some special case.
And so warnings were shown when the source is being tried
to be removed with the already removed id.

Fix this by unconditionally resetting the id on start of the function.

textview: Simplify creating bubble action buttons

Buttons can be created simply with gtk_button_new_from_icon_name().

widget: Avoid an unnecessary check

The preceding loop terminates when either the widget is NULL,
or if their type matches. There is no reason to check that again

wayland: Fix restarting cursor animation

When an animated cursor was set and the previous cursor animation delay
happened to be the same, we wouldn't restart the animation timeout and
just return G_SOURCE_CONTINUE assuming the timer would continue. This
assumption is however only valid if the function was called from the
timeout, which is not the case.

Instead also arm the timer also if there is no previous timer active.

wayland: check native surface for crossing events

gdk_wayland_*_grab()/ungrab() would emit crossing events which translate
as focus_in/focus_out events for keyboard.

However, the ungrab() functions compare the native toplevel as this is
what gets the Wayland pointer enter/leave events with the grab surface,
so if the grab is issued on a child gdk surface, those won't match and
we would emit more focus_out events than focus_in.

This means that a widget such as spice-gtk which issues a keyboard grab
whenever the pointer enters the surface and releases the grab when it
leaves the surface would get uneven numbers of focus_in/focus_out
events.

Also, gdk_wayland_seat_ungrab() would not emit crossing events for
keyboard devices, whereas gdk_wayland_device_ungrab() does, which adds
even more potential discrepancies between focus_in/focus_out events.

To solve this problem, introduce two new helper functions which check
the relevant native surfaces to emit crossing events when needed that
get called evenly from both gdk_wayland_seat_grab()/ungrab() and gdk
_wayland_device_grab()/ungrab() APIs.

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=780422
Closes https://gitlab.gnome.org/GNOME/gtk/issues/792

examples: Use icon-name property to set button icons

Gtk4 added an icon-name property for GtkButton.

examples: Simplify counting buffer lines

We do have a gtk_text_buffer_get_line_count() function
to get the line count.

examples: Remove explicit gtk_widget_show()

In gtk4, widgets are visible by default.

Fix comment typo introduced in a32725bc9ab