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

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

Updated Czech translation

examples: Take advantage of user_data passed

The last parameter of the signal callback from .ui
is the template's object from which the class is
derived.

And so, we already have access to the window object.
Let's just use it.

ci: Disable tests when building Graphene

Our flatpak-builder manifests include building Graphene from Git; since
we're building the GTK demos, it's pointless to build the Graphene tests
as well. Disabling tests and benchmarks avoids pointless installations
inside the Flatpak build repo that will just be removed by the time we
bundle the demo.

docs: List private headers for gtk-doc to ignore

We have a lot of private headers that should not be used by gtk-doc when
scanning for symbols.

Avoid absolute paths when building the API reference

We should always use relative paths, or include_directories objects, to
allow using GTK as a subproject.

Updated Spanish translation

Updated Spanish translation

examples: Don't use private members for final classes

Only derivable classes are required to have private members
so that derived classes can't override them.

reference: Replace reference to gtk3 with gtk4

reference: Use gtk+-4 for compilation in examples

examples: Simplify .ui files

Run gtk4-builder-tool recursively

examples: Fix Makefiles to use gtk4 for build

dnd: Make "formats" a construct-only property

... and hide the member variable inside the DragContextPrivate.

wayland: Remove stray if

Leftover from d056be4ea238e6f4371230b1a892ed0100770848.

I need more sleep when I need to fixup the fixups of my fixups now.

win32: Remove unused file

Merge branch 'wip/sadiq/fixes' into 'master'

icon-browser: Fix typo in description

See merge request GNOME/gtk!138

icon-browser: Fix typo in description

This is shown as tooltip-text

gsk: Improve GSK_RENDERER behavior

1. Include the broadway renderer (so we can test it properly fails on
   Wayland or X11)
2. List all potential renderers, print useful information when Vulkan
   is not compiled in instea dof omitting it
3. Improve docs

wayland: Fix memleak

7f8a8f221d60bc5da23602e5e024eb91f8b0f758 forgot to free the data offer
passed to it.

Update docs for debug environment variables

Just the usual pre-release doc fixups.

Document the GSK_RENDERER environment variable

Just the usual pre-release doc fixups.

Update build docs

Just the usual pre-release doc fixups.

Miscellaneous docs improvements

Just the usual pre-release doc fixups.

Add GtkWidgetPaintable to the docs

Just the usual pre-release doc fixups.

Add GdkPaintable to the docs

Just the usual pre-release doc fixups.

MediaStream: Fix capitalisation of two new %TRUEs

Add more media docs

gitlabci: Build and publish docs with gitlab pages.