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

filechooser: Fix the gridview

See merge request GNOME/gtk!5817

filechooser: Fix the gridview

2a70093a309481cf6 was a bit overeager in its porting
to GtkColumnViewCell. The gridview needs to keep its
list items.

Merge branch 'wip/corey/file-chooser-widget' into 'main'

More fixes to filechooserwidget

See merge request GNOME/gtk!5815

Merge branch 'wip/corey/filesystemmodel' into 'main'

filesystemmodel: Notify correct position removed in remove_file()

See merge request GNOME/gtk!5814

filechooser: Set date and time after cell is a child of filechooserwidget

The date/time column relies on the filechooserwidget to format the date
properly.  During bind, the filechoosercell, get the filechooserwidget
ancestor, but now due to changes in the listview, the cell isn't a
child of the filechooserwidget at that point.  Since this is deeply
ingrained into the filechooserwidget, let's keep the same behavior,
but move it to filechoosercell in realize.  Alternatively, we could have
used a signal factory (with the file chooser widget as the user data),
but that would have been a major overhaul.

filechoosercell: Store type_format in filechoosercell

The format of the type column depends on the the type_format, which
is stored in the filechooserwidget.  We get that setting by looking
for the filechooserwidget ancestor, which no longer works after recent
changes to the list views (it was fragile to begin with).  At one point,
the setting appears to have been dynamic, but now it is only loading
from GSettings, so let's simply do the same within FileChooserCell.

filechooserwidget: Finish port to GtkColumnViewCell

Some missing updates from f5dea9a3c217c49f44eedb18822be080d2451dc0

filesystemmodel: Notify correct position removed in remove_file()

32247bc50e63b709ae738a95a2ec71345b5fad75 made several changes to account for the
fact that we no longer have a NULL editable at the beginning of the list
model.  The commit mistakenly left out one change in remove_file(),
which causes the wrong file to be removed.

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

node-editor: Typo fixes

See merge request GNOME/gtk!5812

node-editor: Typo fixes

Fix some typos in the node format documentation.

Merge branch 'scrolled-window-critical' into 'main'

scrolledwindow: Avoid a critical

See merge request GNOME/gtk!5809

testsuite: Check widget properties better

Check that setting a property after resetting
it works. An instance of this was fixed in the
previous commit.

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.

Merge branch 'warn-show-destoyed-window' into 'main'

Show a warning when a destroyed window is shown

See merge request GNOME/gtk!5122

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

gtk-demo: Add a keyword

See merge request GNOME/gtk!5805

gtk-demo: Stop using gtk_widget_get_allocation

gtk-demo: Add a keyword

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

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

range: Fix a copy-paste error

See merge request GNOME/gtk!5803

placessidebar: Stop using gtk_widget_get_allocation

coloreditor: Stop using gtk_widget_get_allocation

range: Fix a copy-paste error

This was showing up as the alpha popup in
the color editor not being positioned correctly.

Merge branch 'gbsneto/vulkan-glyph-fixes' into 'main'

Vulkan glyph fixes

See merge request GNOME/gtk!5802

gsk/vulkan/glyphcache: Rework glyphs

The current implementation of the glyph cache deals with atlases by
padding them with 1 pixel at the beginning, at the end, and between
each glyph.

That's cool and all, however, there's a very subtle problem with
this approach: the contents of the atlas are garbage, so this padding
is filled with garbage memory!

Rework the Vulkan glyph cache to draw each and every glyph in a
surface that has 1 pixel border of padding around it. Ensure the
surface is completely black by drawing a rectangle before handing
it to Pango to draw the glyph. Update tx and ty to pick the texture
position adjusted to the 1 pixel padding. The atlas now starts at
position (0, 0), since each glyph individually contains its own padding.

To improve legibility, add a PADDING define and use it everywhere.

gsk/vulkan/pipeline: Simplify pipeline creation

Nothing uses gsk_vulkan_pipeline_new_full() anymore.

gsk/vulkan: Use default blend factors

Vulkan renders text using VK_BLEND_FACTOR_ONE_MINUS_SRC_ALPHA and
VK_BLEND_FACTOR_SRC_ALPHA, but that implies per-channel alpha
blending, which currently produces the wrong results when blending
glyphs with the images beneath them.

Use the default pipeline constructors, which implies using the
ONE and ONE_MINUS_SRC_ALPHA.

gsk/vulkan: Cosmetics

Reorder code to match the order used in shaders.

Tweak the wording of the message.

Merge branch 'wip/carlosg/popover-ungrab-on-unmap' into 'main'

gtkpopover: Move GTK grabs to map/unmap

Closes #5730

See merge request GNOME/gtk!5801

gtkpopover: Move GTK grabs to map/unmap

Typically, a popover gets mapped when shown and unmapped when
hidden. A situation there that breaks is where the popover gets
recursively unmapped/unrealized when its root is destroyed.

In that situation, the popover does however unmap (without being
hidden first), moving the GTK grab from show/hide to map/unmap
will handle the previous situations, plus this one.

Fixes things being unclickable if e.g. a modal dialog got a
popover popped up, then got closed via Alt-F4.

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

Merge branch 'inspector-action-fixes' into 'main'

actionmuxer: Make the compiler warn

See merge request GNOME/gtk!5800

inspector: Redo the actions list

Make the actions list implement GtkActionObserver
for the muxer case, and make updates work.

inspector: Simplify the action editor

No need to listen for changes, the actions list
code will do so and tell us to update.

inspector: Add ActionHolder::changed

We will use this signal to communicate
action changes to the action list widgetry.

inspector: Cosmetics

Make the variant editor use a checkbutton
instead of a togglebutton.

actionmuxer: Add a precondition

This helps with debugging the inspectors action
handling.

Update Bulgarian translation

(cherry picked from commit f15aa82eaf4a8d9498b53e3e416d2ac21d6b24f4)

inspector: Be more careful

Tweak the variant-editor code to handle some
cases more explicitly.

inspector: Plug a memory leak

inspector: Be safer when handling actions

Those query apis are serious about not ignoring
their return value, and may not set their our
arguments to anything if they return FALSE.

actionmuxer: Make the compiler warn

Warn when the boolean return isn't used, since we
may not initialize the out arguments in the FALSE
case (see the previous commits).

textview: Handle a return value

Merge branch '5729_fix_inspector_crash' into 'main'

inspector: fix crash caused by uninitialized GVariant

Closes #5729

See merge request GNOME/gtk!5799

inspector: fix crash caused by uninitialized GVariant

we were not checking the return gboolean of
gtk_action_muxer_query_action() which was
returning FALSE for the crash case, meaning
it didn't set the passed in GVariant, but
we were still using it as it was non-null.

Fixes #5729

Merge branch 'media-controls' into 'main'

mediacontrols: Add tooltip text to play button

See merge request GNOME/gtk!5794

Merge branch 'alert_dialog_role_mapping_fix' into 'main'

a11y: Map GTK_ACCESSIBLE_ROLE_ALERT_DIALOG to ATSPI_ROLE_ALERT

See merge request GNOME/gtk!5796

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

glarea: Fix property notification

See merge request GNOME/gtk!5797

glarea: Fix property notification

This was broken in 9c26262e3e083006.

mediacontrols: Add tooltip text to play button

This makes the button accessible.

a11y: Map GTK_ACCESSIBLE_ROLE_ALERT_DIALOG to ATSPI_ROLE_ALERT

By doing this, alert dialogs will be read automatically, or at least
they should be, because a screen reader know that these are special.

Merge branch 'wip/exalm/centerlayout-shrink' into 'main'

centerlayout/box: Add :shrink-center-last

See merge request GNOME/gtk!5790

Merge branch 'wip/exalm/shortcuts' into 'main'

Add style classes to standard dialogs

See merge request GNOME/gtk!5791

pagesetupunixdialog: Add "pagesetup" style class

fontchooserdialog: Add "fontchooser" style class

filechooserdialog: Add "filechooser" style class

colorchooserdialog: Add "colorchooser" style class

appchooserdialog: Add "appchooser" style class

shortcutswindow: Add "shortcuts" style class

centerbox: Expose GtkCenterLayout:shrink-center-last

centerlayout: Add :shrink-center-last

Allow to prioritize start and end children and have them keep their natural
size instead of center child if possible.

See https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/5552#note_1708864

Merge branch 'ebassi/listbox-dispose' into 'main'

listbox: Unbind the model before removing children

See merge request GNOME/gtk!5789

listbox: Unbind the model before removing children

Calling gtk_list_box_remove_all() is a no-op with a bound model; after
the introduction of the remove_all() method in 49e56fc7, we were left
with row widgets after the dispose() call chained up.

We could restore the explicit unparenting inside dispose() instead of
calling remove_all(), but since the bound list model is provided by the
user of GtkListBox, it's more appropriate to unbind it in the dispose()
implementation, to avoid any potential reference cycle (especially in
higher level languages that have no explicit reference acquisition).

We clean up the bound model, and its associated state, if any; and then
we remove all the row widgets that are left.

Merge branch 'glarea-allowed-apis' into 'main'

glarea: Add GtkGLArea::allowed-apis

Closes #5330

See merge request GNOME/gtk!5786

gtk-demo: Update the GLArea examples

glarea: Add GtkGLArea::allowed-apis

Add api that mirrors the new GdkGLContext api
for selecting OpenGL variants, and deprecate
the old use_es api.

Fixes: #5330

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

expander: Fix the css docs

Closes #5723

See merge request GNOME/gtk!5787

Merge branch 'node-editor-things' into 'main'

Revert "rendernodepaintable: Provide accurate aspect ratio"

See merge request GNOME/gtk!5788

node-editor: Improve the scaling

We only want to scale the main rendering, not whats
shown in the sidebar. Also, make the scale logarithmic.

rendernodepaintable: Preserve aspect ratio in snapshot()

This is the better fix.

Revert "rendernodepaintable: Provide accurate aspect ratio"

This reverts commit 52a849288793e7eede878798a88760c058266f60.

It turned out to be the wrong fix.

expander: Fix the css docs

The docs were not matching reality since the
css node names were changed in 6d20fe0b.

Fixes: #5723

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

rendernode: Remove instance_init() function

See merge request GNOME/gtk!5785

Merge branch 'bilelmoussaoui/return-fix' into 'main'

gtk: Fix *_get_tab_behaviour return type

See merge request GNOME/gtk!5784

rendernode: Cosmetics

Simplify a function.

rendernode: Get rid of GskRenderNodeTypeInfo

Instead, use class_init functions.

rendernode: Remove instance_init() function

It's not used.

gtk: Fix *_get_tab_behaviour return type

Post-release version bump

4.11.1

NEWS: Updates

Add comment on versions

Some versions are specfied in multiple places.
Add a reminder to update all places.

Merge branch 'gbsneto/vulkan-layout-fixes' into 'main'

Vulkan fractional scale support, image layout and clip region fixes, cleanups

See merge request GNOME/gtk!5783

vulkan: Support fractional scaling

Basically what GL does, but without any debug or feature flag
to gatekeep it, since the Vulkan backend itself is experimental
already.

Ceil surface sizes, and floor coordinates, to the fractional scale
value.

gsk/vulkan/renderpass: Don't scale scissor and render area

The rects passed to the clip region are in buffer coordinates, and
must not be scaled. Consider the following scenario: Wayland, with
a 1024x768@2 window. That gives us a 2048x1536 raw image. To setup
the Vulkan render pass code, we'd scale 2048x1536 *again*, to an
unreasonable 4196x3072, which is (1) incorrect and (2) really
incorrect and (3) can lead to crashes at best, full GPU resets
at worst - and a GPU reset is incredibly not fun!

Now that we pass the right clip regions at the right coordinates
at all times, remove the extra scaling from the render pass.

gsk/vulkan: Cosmetics

No functional changes. Use a variable that already exists instead
of calling gdk_draw_context_get_surface() again. Cleanup some
newlines.

gsk/vulkan/renderer: Pass appropriate clip region

This part of the Vulkan renderer is almost exactly equal to the GL
renderer, and the GL renderer already does that since at least
2a38cecd3351. Copy that into the Vulkan renderer.

A nice side effect from this commit is that resizing a window now
actually works again.

Sneak in a trivial cleanup by using a variable to hold the draw
index.

gsk/vulkan/buffer: Pass aligned memory value

This was a tricky one to figure out, but it's pretty simple to
understand (I hope!).

So, this AMD card I'm using requires buffer memory sizes to be
aligned to 16 bytes. Intel is aligned to 4 bytes I think, but
AMD - or at least this AMD model in particular - uses 16 bytes
for alignment.

When creating a a particular texture (I did not determin which one
specifically!) a buffer of size 1276 bytes is requested.

1276 / 16 = 79.75, which is clearly not aligned to the required
16 bytes.

We request Vulkan to create a buffer of 1276 bytes for us, it
figures out that it's not aligned, and creates a buffer of 1280
bytes, which is aligned. The extra 4 bytes are wasted, but that's
okay. We immediately query this buffer for this exact information,
using vkGetBufferMemoryRequirements(), and proceed to create actual
memory to back this buffer up.

The buffer tells us we must use 1280 bytes, so we pass 1280 bytes
and everyone is happy, right? Of course not. We pass 1276 bytes,
and Vulkan is subtly unhappy at us.

Fix that by passing the value that Vulkan asks us to use, i.e.,
the size returned by vkGetBufferMemoryRequirements().

gsk/vulkan/glyphcache: Ceil glyph surface size

This is what GL does, and for a reason: it can lead to width or
height for very small glyphs. Also, switch to dividing by a float
(1024.0) instead of an integer (1024).

gsk/vulkanimage: Set buffer row length and height

This doesn't make any difference now, but will allow us to copy
subregions more easily. This is not obvious, but here's a quick
explanation:

Leaving 'bufferRowLength' and 'bufferImageHeight' implies that
Vulkan will assume the size passed in the 'imageExtent' field.
Right now, this assumption is correct - the only user of this
function is the glyph cache, and it only copies and uploads
exact rects. Next commits will change that assumption, so we
must pass 'buffer*' fields, and tell Vulkan, "this part of the
buffer represents an image of width x height, and I want the
subregion (x, y, smallerWidth, smallerHeight) of this image".

gsk/vulkan/image: Use UNDEFINED for initial layout

When creating an image using gsk_vulkan_image_new_for_framebuffer(),
it passes VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL.

However, this is a mistake. The spec demands that the initial
layout must be either VK_IMAGE_LAYOUT_UNDEFINED or
VK_IMAGE_LAYOUT_PREINITIALIZED.

Apparently this was an oversight from commit b97fb751469, since the
commit message even documents that, and all other calls pass either
VK_IMAGE_LAYOUT_UNDEFINED or VK_IMAGE_LAYOUT_PREINITIALIZED.

Create framebuffer images using VK_IMAGE_LAYOUT_UNDEFINED, which is
what was originally expected.

Merge branch 'listbox-remove-all' into 'main'

gtklistbox: Add remove_all()

See merge request GNOME/gtk!5779

Merge branch 'zbrown/bump-protocols' into 'main'

build: We need wayland-protocols 1.31

See merge request GNOME/gtk!5780

Update Russian translation

build: We need wayland-protocols 1.31

https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/5767

gtkflowbox: Add remove_all()

Removing all items from containers is a common use case.
Without this applications needed to implement this manually.
It makes senses to handle it here.

gtklistbox: Add remove_all()

Adds a function to remove all children from
a GtkListBox. This way app developers don't need to
implement this themselves.

gsk/vulkan/image: Cosmetics

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

tooltipwindow: Cast the correct object

See merge request GNOME/gtk!5774

Merge branch 'docs-updates' into 'main'

docs: Document new environment variables

See merge request GNOME/gtk!5777

tooltipwindow: Cast the correct object

the tooltipwindow is in the userdata.

Merge branch 'scale-factor-cleanup' into 'main'

wayland: Be more careful about scale-factor

See merge request GNOME/gtk!5775