MenuButton: Fix small grammar-o/match related syms

menus: Avoid unnecessary right padding

We are using placeholders in the 'check' column
that are put in a size group, so that they all
take the same space once a check or radio is shown.

Unfortunately, for the inline-buttons option, we
were using a GtkBuiltinIcon as placeholder, and those
respect the -gtk-icon-size CSS property and take
a minimum size of 16px. Use a GtkGizmo instead to
get the expected result of no extra padding unless
there's a check or radio.

Fixes: #5839

gtk/recentmanager: Fix date leak

Fix typo in CSS docs

filelauncher: Use ShowItems to show in File Manager

The non-portal fallback method for launching a file manager to show the
file in its parent directory was incorrectly using the `ShowFolders`
method (open a folder) instead of `ShowItems` (open the parent directory
and show the file).

The `show_item` function (previously `show_folder`) had an unused
`callback` parameter; it has been removed and the type of the parameter
containing the GTask has been renamed and now uses the correct type
instead of gpointer to reduce the amount of casting required.

Fixes GNOME/gtk#5842

roaring: Mark a variable as unused

Otherwise clang complains.

scrolledwindow: Propagate child measure size whenever possible

In height-for-width and hscrollbar-policy = never, we can provide
the child with a proper for_size when measuring it. The same is true for
width-for-height and vscrollbar-policy = never.

This allows for accurately measuring the size of eg. wrapping labels.

gsk: Fix a crash

When we start ignoring batches, we must do it everywhere,
or we may run into assertions. This was triggered by an
enormous text node tree produced by tests/rendernode-create.

node-editor: Print default renderer in title

If there is a value passed to GSK_RENDERER, display it in the window
title.

This is mostly so that when I show off screenshots, people know what
renderer I'm using.

rendernode: clip when drawing shadows

Ensures the shadow stays small.

css: Remove crashy resource:// optimization

Using gdk_texture_new_from_resource() is not valid here because we are
not sure if the given resource is valid.

Plus, the previous optimization is no longer relevant, because we are
not using gdk_pixbuf_new_from_resource() anymore - which was what this
optimization was about before it was ported to GdkTexture.

Test attached.

gdk: Clamp frame region to surface size

The region may be larger than the surface's size, but many rendering
APIs require the size to be clamped.

Fixes #5812

Fix a few bugs in GtkPropertyLookupListModel

Fix bugs in property types.

filesystemmodel: Ignore changes on the parent directory

The filesystemmodel tracks changes and additions to child files
through G_FILE_MONITOR_EVENT_ATTRIBUTE_CHANGED.  This event will also
occur if the parent directory is changed.  Since the parent directory
doesn't exist in the model, it creates a non-existent item.

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

filechooserwidget: Stop leaking file system models

doc/actions: Fix end-of-file :

The actions document ended with a : where there used to be a period,
seemingly a bug in the conversion from XML. Replace that : with a
period, and remove another spurious : inserted during the conversion.

docs/input: <kbd>␣</kbd> => <kbd>Space</kbd>

- Switch to semantic newlines, where adding <kbd> caused wrapped text
  to grow wider than 80 characters on some lines.

docs/input-handling: A few more <kbd> wrappings

- Format a few missed keypresses/combos using <kbd>
- Switch to semantic newlines for those (paragraphs, bullet points)

docs/input-handling: Don't leave reader hanging

Strangely, the document's last sentence ended with a colon,
giving the impression there's missing text that should follow.
There is not.

Even more strange, it's _always_ ended with a colon,
ever since the file was converted from XML
(in commit 10cd539104af54f5d1d1e49618f5aa9d8ac42074).

BUT, the XML file it was converted _from_ ended that same sentence
with a period! I have no idea where the colon came from.

Add a README to examples/hello

Just so nobody wonders why there's a meson.build
in this subdirectory.

Widget: Fix doubled word need, missing possessives

popover: Stop making assumptions about corners

The popover tail drawing code was just assuming that
the corner values are numbers. Which they won't, anymore.

css: Fix initial rotate transform value

css: Make some corner values static

Provide static value for uniform corners
with lengths from 0 to 8px. This covers
the majority of corners in widget-factory.

css: More static length values

Make all length values from 0px to 8px available
as static values. This will help with cutting
down on the number of corner values (we have a
lot of 5px corners).

Revert "csscornervalue: Accept other values if x == y"

This reverts commit 1f2f2777e1231468f613a2936989ded060f4e2a6.

It broke transitioning of corner values.

css: Correct initial shadow filter values

This was using numbers instead of lengths,
causing confusion in the transition code.

css: Fix transition of font features

If we store ints as values, we better use
g_hash_table_lookup_extended, or zeros will
show up as missing values.

mediacontrols: Set valign=center on volume button

To match the play button.

Overlay: Add minimal doc blurb for property :child

urilauncher: Validate the uri

We don't want to send nonsense to the portal.

Fixes: #5829

tests: Stop copying the tool tests

No point in doing that, and the meson feature
we are using here is deprecated.

Update Lithuanian translation

Merge branch 'backport_gtkstack_crash' into 'gtk-4-10'

Backport the fix from #6031

See merge request GNOME/gtk!6036

GtkStack: Fix a potential crash in gtk_stack_get_first_accessible_child

This one can occur when the stack has no pages.

Merge branch 'cherry-pick-099955b0' into 'gtk-4-10'

cherry-picking "gridview: Add border-spacing where it was missed"

See merge request GNOME/gtk!6029

gridview: Add border-spacing where it was missed

We were computing column widths without taking border-spacing into
account, making them slightly too big.

(cherry picked from commit 099955b0c06dbddc9469bd3c8e63673a3a82883f)

Update Basque translation

Update Catalan translation

Update French translation

Merge branch 'cherry-pick-29e71868' into 'gtk-4-10'

treelistmodel: Be safer during collapsing

See merge request GNOME/gtk!5918

Merge branch 'wip/otte/treelistmodel-madness' into 'main'

treelistmodel: Be safer during collapsing

See merge request GNOME/gtk!5875

(cherry picked from commit 29e71868298448eb7fa570ed0a75935c235262de)

8766a6fa treelistmodel: Be safer during collapsing
3c76f3fb treelistmodel: Delay notifies from TreeListRow

Merge branch 'wip/carlosg/invalidate-surface-x11-4-10' into 'gtk-4-10'

gdk/x11: Invalidate whole surface after size change (4.10)

See merge request GNOME/gtk!5916

gdk/x11: Invalidate whole surface after size change

The Expose events following a ConfigureNotify may arrive at
a time that we did not resize the surface yet, making these
expose events a no-op. Even though gsk/gtk take care of the
window content itself, this might lead to unrendered portions
of the window shadow.

This may be seen with GSK_RENDERER=cairo and GDK_BACKEND=x11,
attempting to tile a window (e.g. gtk4-demo) left or right.
The window will show black rectangles or other artifacts in
the window shadow areas that correspond to the newly painted
portions (as the window needs to expand vertically).

In order to fix this with a similar behavior to Wayland,
consider ourselves the whole surface invalidated after resize,
in order to ensure everything is painted from scratch.

(cherry-picked from commit 24302315fb8f46be141a9eda60e8d107c84a948d)

Update Persian translation

Merge branch 'backport-css-fix-to-gtk-4-10' into 'gtk-4-10'

Backport "css: Don't transition to currentColor" to gtk-4-10

See merge request GNOME/gtk!5911

css: Don't transition to currentColor

Transition to the color that is in use instead.

Fixes crashes because currentColor is not an RGBA color and
therefor could not be queried later.

Fixes #5798

Update Chinese (China) translation

Update Indonesian translation

Merge branch 'cherry-pick-c5a53f23' into 'gtk-4-10'

Merge branch 'wip/otte/trelistmodel-fixage' into 'main'

See merge request GNOME/gtk!5873

Merge branch 'wip/otte/trelistmodel-fixage' into 'main'

treelistmodel: Don't add items in reverse

Closes #5707

See merge request GNOME/gtk!5744

(cherry picked from commit c5a53f235a2ed1b0acd0a8c29153e62377262d04)

1718db14 treelistmodel: Don't add items in reverse
cd860beb Add a test for splicing treelistmodel

Update Galician translation

Update Turkish translation

Update Swedish translation

Update Ukrainian translation

Update Russian translation

Post-release version bump

4.10.3

Merge branch 'cherry-pick-0ce6bc67' into 'gtk-4-10'

filesystemmodel: Emit items-changed when modifying attributes

See merge request GNOME/gtk!5860

Update Polish translation

Update Slovenian translation

Merge branch 'fix-popover-positioning-4-10' into 'gtk-4-10'

popover: Fix positioning

See merge request GNOME/gtk!5859

filesystemmodel: Emit items-changed when modifying attributes

(cherry picked from commit 0ce6bc677e339b2af213377773e184674cb45c87)

popover: Fix positioning

Update Portuguese translation

Post-release version bump

4.10.2

Move some nonworking reftests to xfail

A few of the inscription reftests have been failing since 4.10.1.
Move them to xfail until somebody shows up to fix them.

Update Georgian translation

Merge branch 'backports-for-4-10' into 'gtk-4-10'

Accumulated backports for 4.10

This branch has my cherry-picks from main since we branched after 4.10.1.
Two inscription reftests are failing, but they were also failing before.

See merge request GNOME/gtk!5856

gdk/win32: drop some unused variables

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk: drop libangle GLES minimum version

GLES 2.0 version is fine now with current gtk according to B. Otte.
Let's use the same minimum requirement for all implementations.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk: use GLES when on win32/ANGLE

When using GDK_DEBUG=gl-egl, we end up using GL, but that is not well supported:

Creating EGL context version 3.0 (debug:no, forward:no, legacy:yes, es:no)
Created EGL context[0000000000000004]
OpenGL version: 0.0 (legacy)
* GLSL version: (NULL)
* Max texture size: -1059701680
* Extensions checked:
 - GL_KHR_debug: no
 - GL_EXT_unpack_subimage: yes
 - OES_vertex_half_float: no

** (gtk4-demo.exe:14324): WARNING **: 19:16:41.468: Compile failure in
vertex shader:
ERROR: 0:7: 'gl_Position' : undeclared identifier
---8<---

Use GLES when EGL implementation is ANGLE.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk: drop unused vertex_array_object

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

glrenderer: Don't try to use float framebuffers on GLES2

GLES doesn't know what that is.

testsuite: Skip focus tests when necessary

If our window does not get focus, we can't reliably
test things that depend on widgets being focused.

Try harder to fix focus tests

The focus test now asserts that its window is active.
So we should not run them in parallel with other tests,
since those might steal the focus.

testsuite: Fix broken test to not unref surfaces twice

The test doesn't hold 2 references, it holds only one.

The reason one unref can cause a leak is that some backends - like X11 -
only destroy the surface once the DestroyNotify event from the X server
has come in.

wayland: Don't leak all surfaces

X11 does add an extra reference to surfaces that gets released when the
DestroyNotify event arrives.
Wayland doesn't ave such an event, so that reference never gets
released.

This fixes a copy/paste error introduced in commit 590f3dfa1fcb.

wayland: Don't insta-crash when a surface gets disposed

We want to remove the event queue from the list of event queues, not the
surface.
Otherwise the freed queue stays in the list and the next time an event
comes in, we access invalid memory.

Fixes thinko introduced in commit 7fafa5133b5.

Luckily, we leak all surfaces, so this problem never occured.

testsuite: Actually wait for focus

We were failing to properly iterate the main
context here. Oops.

tests: Check is-focus instead of has-focus

We are not presenting windows here.

text: Fix fallout from focus changes

We don't get proper notification for when the toplevel
is-active property changes, so monitor has-focus, and
update cursor blinking.

| method                                                                                | current                                                         | suggestion                                                           |
|---------------------------------------------------------------------------------------|-----------------------------------------------------------------|----------------------------------------------------------------------|
| [get_attributes](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L3242) | "use`pango_layout_get_attribute (gtk_label_get_layout (self))`" | "use`pango_layout_get_attributes (gtk_label_get_layout (self))`" [1] |
| [set_yalign](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L5897)     | "(attributes org.gtk.Method.get_property=yalign)"               | "(attributes org.gtk.Method.set_property=yalign)"                    |
| [get_yalign](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L5923)     | "(attributes org.gtk.Method.set_property=yalign)"               | "(attributes org.gtk.Method.get_property=yalign)"                    |
| [set_ellipsize](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L3892)  | "to ellipsizei"                                                 | "to ellipsize"                                                       |
| [get_attributes](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L3239) | "from the labels markup"                                        | "from the label's markup"                                            |

[1] https://docs.gtk.org/Pango/method.Layout.get_attributes.html

| method                                                                                          | current                                                    | suggestion                                                |
|-------------------------------------------------------------------------------------------------|------------------------------------------------------------|-----------------------------------------------------------|
| [Entry/set_invisible_char](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L2101) | "(attributes org.gtk.Method.sets_property=invisible-char)" | "(attributes org.gtk.Method.set_property=invisible-char)" |
| [FlowBox::activate](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkflowbox.c#L541)       | "This can be happen either by"                             | "This can happen either by"                               |

gles: Don't use vertex array objects

They're not needed and GLES doesn't technically support them, even
though GTK had been using them via epoxy sneakily using the
GL_OES_vertex_array_object extension behind our back.

text: Only claim primary when focused

It does not make sense otherwise, and has unfortunate
side-effects on Wayland.

wayland: Improve logging for primary selection

window: Don't mark widget prematurely as has-focus

has-focus is defined is-focus && toplevel::is-active.
We were forgetting to look at is_active when handling
focus widget changes.

gtk: Improve documentation on returned nodes for snapshot

Closes: #5747

openuriportal: Detect if the interface isn't there

Check the portal version number before trying to use
it. Most importantly, this will detect the case where
the interface isn't supported at all, since the proxy
will report a version of 0 in that case.

Fixes: #5733

printunixdialog: Remove a redundant style class

.view does absolutely nothing in Default style since the whole box is
covered with a GtkNotebook which has its own background, and adds an
unwanted background onto the tab strip in Adwaita.

gtk/dialogs: Destroy the window promptly on finish async function

Some bindings (GJS!) could add temporary references to the GAsyncResult
argument that we return, and thus to the GTask, which may cause the
dialog not to close when the finish function is called (but at garbage
collection instead!).

To prevent this, just manually destroy the window (by removing the task
data), so that we are not bound to the GTask lifetime anymore.

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

filechooserentry: Plug a memory leak

filechooserentry: Make filtering work again

We need to look at the filchooser::filtered-out
attribute to know which files the filesystem model
has filtered away.

Fixes: #5743

textview: Reset press counter if double/triple clicking on a different line

If we click close enough between lines, and with the maximum distances applied
by GtkGestureClick we could jump between lines when handling double/triple
click for word/line selection.

Ensure that the whole operation stays in the same line and reset the
gesture/counter if we do move between lines, so we start from scratch in the
new line.

scalebutton: fix orientation not applied to scale

scalebutton: fix CSS name in documentation

scrolledwindow: Avoid a critical

When setting the child property to NULL, we also need
to unset auto_added_viewport, to avoid triggering a
critical when setting it again.

gtk-demo: Add a keyword

Make the demo using the 'bluroverlay' come up when you type 'blur'.