wayland: Don't crash on cursor size 0

The cursor-theme-size setting is documented as
'0 means the default size'. Make it so by using
size 24 if we see a 0. Its better than crashing.

Fixes: #5700

Merge branch 'deprecate-translate-coordinates' into 'main'

Deprecate gtk_widget_translate_coordinates

Closes #5697

See merge request GNOME/gtk!5717

Drop gtk_widget_get_surface_allocation

This private api is confusing, and no longer used.

entrycompletion: Stop using gtk_widget_get_surface_allocation

We only need the width of the widgets' bounds
here anyway, so just use gtk_widget_compute_bounds.

popover: Stop using gtk_widget_get_surface_allocation

The function is going away, and the computation
here was wrong anyway. Instead, add a helper that
properly computes the pointing-to rect in surface
coordinates and use it everywhere.

Deprecate gtk_widget_translate_coordinates

Update all (non-deprecated) callers to use
gtk_widget_compute_point instead.

Fixes: #5697

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

testsuite: Add some GL texture tests

See merge request GNOME/gtk!5715

Merge branch 'popover-bounds-trouble' into 'main'

textview: Give the magnifier correct coordinates

Closes #5695

See merge request GNOME/gtk!5716

magnifier: Correct position of area

The widget paintable uses the widgets bounds
as intrinsic size, so we need to offset from
that to the allocation, which is what the
coordinates are relative to.

texthandle: Correct placement of handles

Text handles had the same problem as popovers.
They were interpreting their pointing-to rectangle
relative to the widgets bounds, when it is meant
to be relative to the widgtets allocation.

While we touch this code, rewrite it to use
gtk_widget_compute_point.

popover: Correct placement of popovers

When we don't have a pointing-to rectangle, we want to place
the popover wrt to the parents bounds. But if we have a
pointing-to rectangle, it is relative to the widgets allocation,
which is different from the bounds.

We were not handling the second case correctly, leading to context
menus in the text view being mispositioned by the widgets CSS padding.

While we are touching this code, rewrite it to handle transforms.

Fixes: #5695

testsuite: Add some GL texture tests

Tests creation and upload/download of GL
textures in various contexts.

testsuite: Cosmetics

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

Fix up criticals from GdkGLTexture

See merge request GNOME/gtk!5712

Merge branch 'alternative-fix-5685' into 'main'

GDK/Win32: Use wgl* functions directly as needed (fix #5685)

Closes #5685

See merge request GNOME/gtk!5702

Fix up criticals from GdkGLTexture

I broke this in 4291f27b562f58.
I overlooked that there may not be a current context.

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

docs: Some updates

See merge request GNOME/gtk!5711

docs: Improve markup for keys

Consistently use <kbd>x</kbd> to render keys in the docs
and use + for key combinations.

docs: Some updates

Fix some links ot point to docs.gtk.org.

inspector: Cosmetics

gtkgstsink.c: Drop workarounds needed for Windows

Since we are making GdkGLContext call the core wgl*() functions directly
instead of via libepoxy, drop the workarounds that we needed for notifying
libepoxy that wglMakeCurrent() outside of GDK/GTK was called.

This way, we clean up the code, and as a result, we can use the GstGL
APIs like the other platforms to query what GL api that is to be used.

For ensuring that things work between different threads, we now call
gdk_gl_context_clear_current() in place of calling wglMakeCurrent(xxx,
NULL), so that we make sure that there is no current GL context on a
thread outside of GstGL's thread, which Windows does not like.

GDK-Win32: Add wrapper functions for calling core wgl* functions

We might be dealing with GL contexts from different threads, which have more
gotchas when we are using libepoxy, so in case the function pointers for
these are invalidated by wglMakeCurrent() calls outside of GTK/GDK, such as
in GstGL, we want to use these functions that are directly linked to
opengl32.dll provided by the system/ICD, by linking to opengl32.lib.

This will ensure that we will indeed call the "correct" wgl* functions that
we need.

This should help fix issue #5685.

Merge branch 'wip/otte/for-main' into 'main'

listview: Activatable means activate

See merge request GNOME/gtk!5709

columnview: Don't infloop when a cell is grab_focus()ed

Whoopsie.

Broken since 30d4629043f.

listview: Activatable means activate

Fix a typo that prevented activatable widgets from activating since the
refactoring in 608be08b0d6.

Merge branch 'mipmap-tracking' into 'main'

gsk: Avoid copying GL texture in more cases

See merge request GNOME/gtk!5704

gsk: Avoid copying GL texture in more cases

When the GL texture already has a mipmap, we don't
have to download and reupload it to generate one.

We differentiate the handling for texture scale nodes,
where we do want to force the mipmap creation even if
it requires us to reupload the GL texture, and plain
texture nodes, where we just take advantage of a
preexisting mipmap to allow trilinear filtering for
downscaling, or create one if we have to upload the
texture anyway.

gl: Add a private way to query mipmap status

Make GdkGLTexture determine if the texture has
a mipmap, and provide private API to query this
information.

This check is done in gdkgltexture.c instead of
gskgldriver.c, since we're already binding the
texture here for other reasons, so it is easy
to query a few more things.

Merge branch 'check-widget-on-destruction' into 'main'

gtkwidget: Do a type-check in gtk_widget_in_destruction()

See merge request GNOME/gtk!5708

gltexture: Try harder to determine the format

We can query these things as long as the texture ID
is valid in the current context.

gtkwidget: Do a type-check in gtk_widget_in_destruction()

Given it can be called during a widget destruction, it's safer to do a
type-check before crashing.

See: https://gitlab.gnome.org/GNOME/gnome-control-center/-/merge_requests/1727

Update Chinese (China) translation

Merge branch 'fix-a11y-leak2' into 'main'

Don't leak all AT contexts

Closes #5690

See merge request GNOME/gtk!5700

Merge branch 'wip/otte/listview-cleanups' into 'main'

listview refactoring and new stuff

See merge request GNOME/gtk!5701

columnview: Handle focus movement properly

- grab_focus() on a row (happens with scroll_to()) keeps the focus
  column intact if possible.
- <Tab> and <Shift-Tab> move through the cells in order, and move
  to the next row when at the end.
- <Up> an <Down> move to the next/prev row, keeping the same column
  focused.
- <Left> and <Right> move to the prev/next cell, if one is available.
  If not, they stay where they are.

columnview: add a (private) focus-column

Make setting the focus column scroll to it, too.

columnviewrowwidget: Handle left/right arrow keys

Make it move focus just like GtkWindow would.

Otherwise the listview will (try to) handle it and move focus between
cells - which doesn't do anything for lists and only works with grids.

listbase: Only grab_focus() if the cursor moved

If there is no movement, grab_focus() on the already focused widget
just causes weird inconsistencies and we don't want those.

columnviewrow: Handle focus movement properly

Handle backwards tab properly and deal with left right arrows.

listitemwidget: Only take a single child

columnviewrowwidget: Drop unneeded stuff

Also remove the GtkColumnListItemFactory and GtkColumnViewLayout
objects by folding them into GtkColumnViewRowWidget.

columnview: Add GtkColumnViewRowWidget

This is a copy/paste of GtkListItemWidget for now.

Modifications will happen in future commits.

listview: Add private GtkListFactoryWidget

This is a split of GtkListItemWidget into the generic parts of factory
using widgets.

On top of it there's GtkListItemWidget, which takes care of GtkListItem.

listview: Move ::factory out of the item manager

It's a listitemwidget property, and the item manager manages the items,
not the widgets.

listview: Move ::single-click-activate out of the item manager

It's a listitemwidget property, and the item manager manages the items,
not the widgets.

listitemmanager: Create widgets via vfunc

We now have listitemmanager just create GtkListItemBase objects via
vfunc and use them.

listitemwidget: split out GtkListItemBase

There's 2 reasons for this:

1. Adding a custom widget for columnview rows
2. Using this as base class for sections

itemmananger: Remove the user data

Use the widget instead. They're the same anyway.

listbase: Privatize 2 functions

They're not used outside of GtkListBase, so no use to have them in the
header.

Requires moving one function up in the source now that the forward
declaration is missing.

listitemmanager: Clean up header

Remove all unused functions and an unused macro.

Don't leak all AT contexts

create_at_context was confused - it stored a reference
to the newly created context in priv->context, but then
also returned a reference, and the caller stored that
in priv->context again.

Change it to only return a reference.

Fixes: #5690

Merge branch 'texture-scale-test' into 'main'

gsk: Add a test for scaled textures

See merge request GNOME/gtk!5686

Update British English translation

(cherry picked from commit d80cecec06c5196bad93be38cc11db86715cca77)

Merge branch 'ebassi/ensure-file-chooser-types' into 'main'

Ensure GtkFileChooserCell's type

See merge request GNOME/gtk!5696

Ensure GtkFileChooserCell's type

Otherwise anything that dlopen()'s GTK will have a bad day.

See: https://discourse.gnome.org/t/strange-filechooserdialog-behaviour-in-gtk-4-10/14435

gsk: Improve slice handling

Upload slices in chunks, instead of copying
enormous amounts of memory.

gsk: Add more flexible texture upload

Add a command queue function to create a
GL texture by uploading several GdkTextures
at given positions.
This lets us 'piece together' textures.

gsk: Tweak one test to pass

Clip away the problematic corners in this
artificial texture scaling test. The GL renderer
has problems getting them perfect.

gsk: Drop debug code

gsk: Drop some unused arguments

All callers of the slice apis pass 0 for
min_rows and min_cols. So just drop these.

gsk: Use overlapping slices

Store texture coordinates for each slice
instead of assuming 0,0,1,1, and generate
overlapping slices to allow for proper mipmaps.

This almost fixes trilinear filtering with
sliced textures.

gsk: Add a test for scaled textures

GL and cairo disagree on this one, and have forever.
Whee!

gsk: Cosmetics

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

a11y: Add a test for gtk_accessible_role_to_name

See merge request GNOME/gtk!5694

a11y: Add a test for gtk_accessible_role_to_name

This should prevent !5693 from happening again
in the future.

Merge branch 'fix_gtk_accessible_role_to_name_crash' into 'main'

fix crash with gtk_accessible_role_to_name and GTK_ACCESSIBLE_ROLE_TOGGLE_BUTTON

See merge request GNOME/gtk!5693

fix crash with gtk_accessible_role_to_name and GTK_ACCESSIBLE_ROLE_TOGGLE_BUTTON

presumably since:

commit de80f503e4db282a7343aa16d2803967d4179739
Date:   Thu Feb 23 14:22:20 2023 +0100

    a11y: Introduce a dedicated role for toggle buttons

Merge branch 'wip/otte/for-main' into 'main'

checkbutton: Only andle arrow focus if we're in a group

See merge request GNOME/gtk!5692

testsuite: Add a treelistmodel test

This one exhaustively tests reusing the same model as a child model for
many nodes.
This tracks that multiple items-changed signals emitted at the same time
(or multiple handlers for one such signal) doesn't put the treelistmodel
in an inconsistent state while it is handling all of them.

I'm not sure this (ab)use of treelistmodel should be officially
supported, but it works today, so let's test it to see if we can keep it
working.

checkbutton: Only andle arrow focus if we're in a group

If there is no other widget in the group that we can focus, don't focus
and activate ourselves.

Otherwise the arrow keys on checkbuttons toggle the checkbutton.

Merge branch 'disabled-picture-opacity-fix' into 'main'

theme: Use 0.5 opacity for disabled pictures

Closes #5683

See merge request GNOME/gtk!5689

theme: Use 0.5 opacity for disabled pictures

If a GtkImage is using an icon we use a gtk-icon-filter to se the icon
opacity when in insensitive state, however when using other kinds of
pictures we do not apply the same style leading to an inconsistent
result.

Closes: #5683

Merge branch 'alatiera/fedora-bump' into 'main'

ci: Use Fedora 38 as the base

See merge request GNOME/gtk!5688

vulkan: Update error codes enum

ci: Use Fedora 38 as the base

Merge branch 'bilelmoussaoui-main-patch-49195' into 'main'

g-i: Add since annotation to GdkSurfaceSize

See merge request GNOME/gtk!5687

g-i: Add since annotation to GdkSurfaceSize

Merge branch 'mipmap-more' into 'main'

gsk: Use mipmaps when it is beneficial

See merge request GNOME/gtk!5685

gsk: Use mipmaps when it is beneficial

If a texture node ends up getting scaled
down a lot, it helps to use trilinear filtering.

Merge branch 'wip/otte/for-main' into 'main'

treelistmodel: Clear item in right place

Closes #5681

See merge request GNOME/gtk!5682

treelistmodel: Clear item in right place

Items should be cleared when the node is discarded, not when the node's
children are discarded - which can also happen when a node is collapsed.

Fixes an error introduced in 9048e391b60e19ccefa0b9fe51c466d16b42dcc4

Fixes #5681

Merge branch 'texture-scale-tests' into 'main'

node-editor: Embed textures in testcases

See merge request GNOME/gtk!5681

ci: Make reports for all setups

testsuite: Improve setup name consistency

Rename the 'waylandgles' test setup to 'wayland_gles'.

ci: Run tests with small textures

testsuite: Add a test setup for slicing

Add a 'wayland_smalltexture' test setup that forces
the max texture size to be 1024, which lets us test
the texture slicing codepaths.

testsuite: Add some texture tests

In constrast to our other tests, these use
textures that are big enough to force slicing
with setting GSK_MAX_TEXTURE_SIZE, which we
will use in the following commits to improve
test coverage.

node-editor: Embed textures in testcases

It is useless to have node files with references
to external files in the testsuite, so turn such
textures into data urls by doing a serialization
roundtrip.

Merge branch 'wip/otte/for-main' into 'main'

Fix inspector doing weird stuff

Closes #5646

See merge request GNOME/gtk!5678

treelistmodel: Clarify docs

No user knows that we have an internal function called
gtk_tree_list_row_destroy() that gets called when a row gets removed
from the treelistmodel.

So everyone was probably just making stuff up about what "destroy"
means.

Related: #5646

treelistmodel: Track the item in the row

That way, we can return the item even after the row is removed. This is
particularly relevant in ListItemFactory::unbind callbacks because they
often use gtk_tree_list_row_get_item() and user code never tracks
changes to this property.

A side effect of this is that the item will survive until the row gets
destroyed, but that's what users expect anyway, so we can live with it.

Related: #5646

treelistmodel: Cache the item

This is a good idea to avoid reentrancy problems when any child model
(or potentially more than one child model) has started emitting
items-changed but the emission hasn't arrived in this model yet.

At tat point, we'd get_item() the wrong item from those models.

We want to avoid such cases of reentrancy.

Related: #5646

cssnode: Refer the correct previous item

We disconnect from old_previous, not from the new previous.

Related: #5646

Merge branch 'wip/matthiasc/samplers' into 'main'

glrenderer: Handle filters differently

See merge request GNOME/gtk!5676

glrenderer: Support GL without samplers

We cheat and just set the texture parameters instead and hope nothing
explodes.
So far it didn't.

This is only needed to support GLES 2.0 so it's quite a limited set of
hardware these days.

glrenderer: Handle filters differently

Instead of uploading a texture once per filter, ensure textures are
uploaded as little as possible and use samplers instead to switch
different filters.

Sometimes we have to reupload a texture unfortunately, when it is an
external one and we want to create mipmaps.

Merge branch 'docs-tweaks' into 'main'

dropdown: tweak the docs

See merge request GNOME/gtk!5677

docs: Cosmetics

docs: Tweaks

Tweak the wording in the list widget overview to
fix some typos and make things read more smoothly.

dropdown: tweak the docs

Add a link to the list widget overview.