MenuButton: popover connects to menu_deactivate_cb

This was missed (even before my recent patches).

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

MenuButton: Disconnect newly added signal handler

Otherwise, we do stuff we shouldn't, as the failing pipeline showed:
https://gitlab.gnome.org/GNOME/gtk/pipelines/9431

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

MenuButton: Clarify NULL popup/over/menu/model doc

MenuButton: Drop ref to Popover on its ::destroy

Otherwise, if the Popover is destroyed before the MenuButton, the latter
still had a non-NULL but invalid instance and tried to use it in dispose

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

Entry: Show optional style classes in node diagram

a11y/ScrolledWin|IconView: Connect signals safely

Use g_signal_connect_data() instead of g_signal_connect_object()
to make sure the callback gets disconnected when the data object
is destroyed. This avoids problems in garbage-collected bindings.

https://bugzilla.gnome.org/show_bug.cgi?id=789215

Update Polish translation

widget: Position changes don't require a redraw

Well, they don't require a redraw of the widget, because the widget
itself didn't change.
They require a redraw of the parent, because that now displays the
widget in a different position.

And this means we can keep the cache of the widget's render node.

My fishbowl numbers are through the roof^W water surface. Vulkan gets
4000 now.

demo: Don't add more icons all the time

If adding new icons takes longer than a second, the frame clock should
run before we add even more icons...

build: Remove 'name' kwarg from has_function calls

They cause warnings with later meson versions.

build: Fix copy/paste error

Always use #ifdef for G_ENABLE_DEBUG

Release builds don't define it to 0, they just don't define it. Avoid
the compilation warning we get otherwise.

sizerequestcache: Directly initialize some locals

Declaring them and initializing them in the same block of code without
an empty line in between is really just confusing.

Use inlined gtk_widget_get_display version in a few places

We can safely do this whenever we already made sure that the given
pointer is a valid GtkWidget.

widget: Add static inline version of gtk_widget_display()

Due to the few type checks in gtk_widget_get_display(), it was the
slowest part of a call to gtk_widget_query_size_for_orientation if the
in case of a cache hit.

textview: Remove handling_key_event flag

It's not used anymore.

gsk: Make gsk_text_node_new_with_bounds private

We pulled out the bounds calculation for performance reasons, but the
caller can't know how to properly compute them. Inside gtk+, we can do
that but it's not good enough for public API.

gl renderer: Rename texture_id parameter to fbo_id

So this makes sense again.

widget: Use get_instance_private more often

To be more consistent with newer code. We can also save a few lines here
and there by pulling the priv initialization before the precondition
checks.

sizerequestcache: Constify some locals

Make it clear that we only use them for lookups here.

sizerequestcache: Use int instead of gint

sizerequest: Pull locals in closest scope

snapshot: Fix some documentation problems

viewport: Remove useless import

Just a debugging remnant

viewport: Remove outdated comment

gl renderer: Fix fallback node scaling

Merge branch 'print-deserialize' into 'master'

printing: Be more careful when deserializing

See merge request GNOME/gtk!121

printing: Be more careful when deserializing

The GVariant we are getting here might not be coming
from GTK+, but rather from some other source. Best to
be forgiving and deal with missing data without crashing.

This was causing the GTK+ portal backends to crash on
print requests from Qt.

Updated Spanish translation

Revert "Update Spanish translation"

This reverts commit 8570d33e08a72abf9004a92784663a3aeba86c00.

stack: protect set_visible_child_name from NULL stack

Return with error if gtk_stack_set_visible_child_name is called
with NULL parameter

(cherry picked from commit 2ee5aee4a9f01cce3dda7dff877070b62c3e2880)

Merge branch 'gtkplacesview-finalization-fixes-master' into 'master'

Gtkplacesview finalization fixes

See merge request GNOME/gtk!119

gtkplacesview: disconnect from server list monitor changes on destroy

It might happen otherwise that a change is recorded in between the
widget dispose and finalization, causing a crash when setting
the visible name for the GtkStack (as that will be NULL at that point)

gtkplacesview: unset entry_pulse_timeout_id before removing it

Fixes a warning on widget finalize, when trying to remove an invalid
source.

Scale: Document new optional classes on value node

Range: Make down/up keys act like down/up scrolls

Before now, down/up keys on H Ranges would increase/decrease value resp,
which is unintuitive & worse, contradicts what we already do for scrolls

Fix simply by moving to the new should_invert_move() as scrolls just did
– which also gets us the other benefits explained in the last 2 commits.

https://bugzilla.gnome.org/show_bug.cgi?id=407242
https://bugzilla.gnome.org/show_bug.cgi?id=791802

Range: Use should_invert_move() to scroll value

This fixes RTL and/or :inverted Ranges responding to a horizontal scroll
by moving the value/slider button in the opposite direction... See prev.

https://bugzilla.gnome.org/show_bug.cgi?id=791802

Range: Add should_invert_move() for scrolls & keys

This will be used in subsequent commits to fix the sign by which the
value is changed in response to directional scroll or keypress events.

The idea is: you have a movement to make – in the form of a delta that
follows widget directions, i.e. −1 means left or up, +1 means right or
down – and you want to know whether that delta needs to be inverted in
order to produce the intuitively expected directional change of :value.

The existing should_invert() is not sufficient: it just determines
whether to invert visually, but we need more nuance than that for input.

To answer that – while not doubling up the work for scrolls and keys – I
add a helper should_invert_move(), which considers other relevant state:

 • A parallel movement on priv->orientation should just use the existing
   should_invert(), which already worked OK for this case (not others).

 • Movements on the other orientation now depend on priv->orientation:

    ◦ For a horizontal Range, always invert, so up (i.e. −ve in terms of
      widget coords) always means increase value & vice-versa. This was
      done in get_wheel_delta(), but move it here for use with keys too.

    ◦ For a vertical Range, ignore :invert as it’s only relevant to the
      parallel orientation. Do not care about text direction here either
      as RTL locales do not invert number lines, Cartesian plots, etc.

This returns TRUE if the delta should be inverted before applying to the
value, and we can now use this function in both scroll and key handlers.

https://bugzilla.gnome.org/show_bug.cgi?id=407242
https://bugzilla.gnome.org/show_bug.cgi?id=791802

snapshot: Don't handle clip anymore

If widgets want to clip things, they now need to do it themselves.

By not taking care of clip, we avoid the need to track clip. And by not
tracking clip, we can avoid all unnecessary cache invalidations that we
were doing for render nodes whenever the clip changed.

And when you are scrolling, the clip changes *a lot*.

MenuShell: Avoid compiler warning re un/signed cmp

priv->button is a guint, but we assigned it to a local gint.

gtk/gtkmenushell.c:734:37: warning: comparison between signed and
  unsigned integer expressions [-Wsign-compare]
           if (button && (new_button != button) && priv->parent_menu_shell)
                                     ^

Menu|Item: Fix FIXME re non-const interned strings

These are members of the private struct, so it hurts no one to fix this.

Menu: Remove redundant typecheck in private func

Everything that calls this already ensured we are a valid GtkMenu.

Menu: Update @accel_path annotation/description

Menu: Typecheck AccelGroup passed via public API

Menu: Typecheck before dereferencing to get ->priv

MenuShell: Sanitise take_focus bool via public API

We store in priv then compare it later, so better make sure it’s 0 or 1.

MenuShell: Typecheck the instance @ select_first()

since this is public API and therefore might get junk passed to it.

CellRenderer: Link to CellEditable.start_editing()

...from CellRenderer::start-editing, to point people in the direction of
info about the lifecycle of the Editable and how to do generic setup.

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

CellRenderer: Clarify doc of .start_editing()

Drop the line copied from .activate(), replace it with a description of
what this method actually does, and explain what a NULL result means.

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

CellRenderer: Entry is not the only CellEditable

so link to GtkCellEditable and note that GtkEntry is just one example.

CellRenderer|Editable: Don't @See_also subclasses

The documentation knows that from the inheritance hierarchy, without us
having to manually duplicate that info here. Link to each other instead.

CellEditable: Clarify doc on lifecycle of editable

* Note in the intro that we're really thinking about temporary widgets
* Mention a gotcha regarding GtkEntry and how ::focus-out stops editing
* Give some examples of what you'd want to do in ::editing-done
* Be a bit more precise about what ::remove-widget represents
* Summarise the lifecycle between Renderer/Editable in .start_editing()
* Emphasise again there that this should be viewed as a temporary widget

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

CellEditable: Move arg description to right place

Move it from the body to the argument line, and while there, update
the deprecated (allow-none) to (nullable).

CellEditable: Fix grammar error in doc synopsis

ci: move the docker image to gitlab.gnome.org

Update Spanish translation

Add an issue template for crashers

Better to have separate templates than a large, "choose your own
adventure" one.

Update Croatian translation

Merge branch 'code-owners' into 'master'

docs: Start defining the owners of code sections

See merge request GNOME/gtk!116

docs: Start defining the owners of code sections

When filing a new merge request it's often hard to know who to ask for a
review; using the Git log doesn't always help — the person that touched
a file last may just be fixing the build or a compiler warning.

The `CODE-OWNERS` file format is something that GitHub uses in order to
pre-fill the list of reviewers:

  https://help.github.com/articles/about-codeowners/

Ideally, in the future, we'll be able to use this file with a bot like
homu to automatically go through newly filed merge requests and
automatically ask the relevant people for reviews, instead of doing this
manually.

Update Croatian translation

ci: remove allow_failure from flatpak jobs again

It was added in 814b7d7bb71920a due to too old wayland in the runtime.
5b160efc768d3 fixed it by bundling a newer wayland.

No need to clear GTK_MODULES anymore

Not a variable we care about nowadays.

Spruce up config summary

Show all the options. Nothing to hide.

flatpak: Bundle wayland for now

We need a newer wayland than is available in the runtime
currently, so use the power of flatpak and bundle it.

tooltip: Ignore events with pressed buttons

We dont' want to show tooltips in those cases.

gl renderer: Remove unused viewport member

frame: Use underscores in widgets ids in the docs

They will work in composite widget templates this way.

frame: Remove priv pointer

frame: Stop saving label and child allocation

We don't use them for anything anymore.

Remove gtk_render_icon_surface

As part of removing all the cairo_surace_t usage. There's still
gtk_render_icon for the same purpose that takes a GdkTexture*

widget: Use _set_has_tooltip everywhere

real_set_has_tooltip sounds like it's a vfunc implementation, but it
wasn't. The force parameter was also useless so just remove that.

widget: Use priv pointer directly

widget: Update docs to mention ::draw less

Revert "ci: Enable the a11y test suite"

This reverts commit 968a8e1cf0c5bd21e856204d3aa1f4256d7ff814.

things still fail in the gitlab runner

font chooser: Allow tweaking fractions

This makes some sense, and frac is meant to be set globally.

ci: Enable the a11y test suite

The accessibility test suite only performs internal validation.

tests: Update the expected a11y dump results

Keep in sync with the current tree.

The changes are mostly caused by updates in the internal hierarchy of
composite widgets, and the fact that the order in which the widget tree
is traversed is not exactly stable.

tests: Split out the accessibility dump tests

Instead of having a single massive test running through the a11y
directory, we can split off each individual file into its own unit.

Having individual units has several advantages:

 - units are executed in parallel
 - it's easier to identify the failing units
 - logs for failed units are easier to read

ci: Use a common section for Flatpak jobs

Avoid repeating blobs of YAML manually, and use the proper YAML
mechanism to copy-paste blobs instead.

GskRenderer: Plug a memory leak

flowbox: Remove unnecessary queue_draw call

paned: Remove outdated comment

The cursor is not updated in state_flags_changed these days.

build: Print summary last

gl renderer: Don't use texture ids in glBindFramebuffer calls

snapshot: don't create offset nodes inside offset nodes

It's pretty easy to make that one offset node.

gl renderer: Increase GArray size by 2 up front

Since we know we will add two elements anyway.

widget: Avoid a dead assignment

tooltip: Avoid some possibly uninitialized values

po: fix zh_CN translation

wayland: Don't freeze the frame clock too early

We can't freeze the frame clock on commit, but only after-paint,
otherwise the frameclock will resume in the paint stage.

So freeze the frame clock at the end of the frame if we are waiting for
a frame callback.

Note; The diff is only lage because of indentation changes due to
avoiding early returns in favor of a branch.

gdk: Remove gdk_surface_create_similar_image_surface()

It's unused.

gdk: Remove gdk_cairo_surface_create_from_pixbuf()

It's unused and people should use textures and snapshots anyway.

widget-factory: Use a pixbuf instead of a surface

This gets rid of the last user of
gdk_cairo_surface_create_from_pixbuf().

testgtk: Don't set cairo surfaces as icons

The code expects textures these days, so use those.

gdk: Remove gdk_cairo_get_drawing_context()

It's unused.

tests: Use gdk_texture_save_to_png() where appropriate

texture: Add gdk_texture_save_to_png()

It's needed for debugging Timm's code, so better have it here than
hidden in my random-patch vault.

drawingcontext: Remove unused APIs

ci: allow flatpak jobs to fail for now

We can easily revert this when things are fixed again.