file chooser dialog: Put save entry into header

When we have a header bar, put a save entry there and
make the file chooser use it.

file chooser widget: Allow external save entry

Prepare the file chooser to use an external entry in
save mode, instead of the builtin one.

file chooser: Update sidsebar when entry goes away

file chooser: Use icon for create folder button

This follows the latest mockups.

file chooser dialog: Show a subtitle

Use the new "subtitle" property of the file chooser widget
to display a subtitle in the headerbar, if we have one.

file chooser widget: Expose a "subtitle" property

Give the file chooser widget a "subtitle" property, which
contains a short string that explains the current mode.

places sidebar: Add private api to get title

The places sidebar already does all the work to determine the
right label to use for the current location of the filechooser.
We want to use the same label in the headerbar subtitle, so
add some private API to get it.

file chooser: More search rework

Drop the radio buttons for current location vs everywhere, and
always search in the current location.

entrycompletion: no need to return boolean

The boolean _gtk_entry_completion_resize_popup's return
value is not used anywhere, and only adds more complexity
for the method.

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

mir: fix mir pointer event to gdk translation

mir: ensure parent surface before creating child surface

ignore size inc when maximized/fullscreen

Under Wayland, fullscreen/maximized windows may not cover the entire
area when a size increment is specified.

Ignore size increments for fullscreen/maximized windows just like most
window managers do under X11 so that windows with size increments can
still be fullscreen or fully maximized under Wayland as well.

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

drawingarea: checks parent's draw() before call it

commit 6786c9d7 introduced a bug when a drawing area is used for
rendering videos, since GtkWidgetClass doesn't set a draw()
method by itself.

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

Fix warnings in the font options code

This code gets called before a screen is set.

Improve the font options support

Instead of just overwriting the default options, merge the per-widget
font options with the defaults.

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

Add a testcase for font options

GtkSwitch: fix a reentry issue

The introduction of state broke some users which relied on
being able to set active in a notify::active handler.

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

gtkwindow: Reset maximize / fullscreen_initially on state changes

Since 740bcf5, we use these properties to properly compute shadow widths
for unmapped windows. If a client calls gtk_window_maximize and a window
manager unmaximizes a window, we should draw borders, so we need to
reset these when we get the property notification.

window: don't override background every style_updated()

This queues an unnecessary resize on the toplevel, and is not needed
anymore, now that GtkWidget does not call
gtk_style_context_set_background() on the window's GdkWindow anymore.

stylecontext: deprecate gtk_style_context_set_background()

Nothing in GTK calls it anymore.

widget: don't call gtk_style_context_set_background()

We don't need to do this here anymore, since widgets typically render
their backgrounds now.

layout: add a comment for gtk_style_context_set_background()

And wrap it with G_GNUC_IGNORE_DEPRECATIONS.
Unfortunately we can't stop rendering the background altogether here.
Also, refactor some common code in a single function.

fixed: add a comment for gtk_style_context_set_background()

And wrap it with G_GNUC_IGNORE_DEPRECATIONS.
Unfortunately we can't stop rendering the background altogether here.
Also, gtk_style_context_set_background() should really be called every
time the style is updated. Fix that.

window: don't call gtk_style_context_set_background()

socket: don't call gtk_style_context_set_background()

toolitemgroup: don't use gtk_style_context_set_background()

textview: don't call gtk_style_context_set_background()

revealer: don't call gtk_style_context_set_background()

toolpalette: don't call gtk_style_context_set_background()

viewport: don't call gtk_style_context_set_background()

treeview: don't call gtk_style_context_set_background()

plug: don't call gtk_style_context_set_background()

paned: don't call gtk_style_context_set_background()

overlay: don't call gtk_style_context_set_background()

offscreenwindow: don't call gtk_style_context_set_background()

menushell: don't call gtk_style_context_set_background()

menu: don't call gtk_style_context_set_background()

listbox: don't call gtk_style_context_set_background()

iconview: don't call gtk_style_context_set_background()

flowbox: don't set gtk_style_context_set_background()

eventbox: don't call gtk_style_context_set_background()

drawingarea: don't call gtk_style_context_set_background()

Updated Portuguese translation

Updated Portuguese translation

broadway: Implement maximization

x11: Query pointer devices' scroll valuators on toplevel enter events

We used to "invalidate" scroll valuators, so the next scroll event could
be used as the base for the next scroll deltas. This has the inconvenience
that it invariably consumes the first event received after enter and,
due to interactions with WM overeager passive button grabs, there's a
possibility we don't scroll at all if we receive interleaved "smooth
scroll" XI_Motion events and XI_Enter events (Normally triggered by regular
scroll wheels in mice).

In order to fix this, and at the expense of some sync-call overhead on
XI_Enter events (one XIQueryDevice call per slave device), query the
current scroll valuator state for all the slaves of the entered pointer,
so we do know beforehand the right base values. If new devices are plugged
while the pointer is on top of the client, the initialized scroll values
will match the valuators'.

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

placessidebar: Do not set as a "track motion" DnD site

It has the unintended effect of picking the suggested action from the
context, which conflicts with the (possibly 0) status we set on our
::drag-motion handler.

Given this widget is not interested in listening to DnD from every
possible target, it is safe to just disable it.

GtkWindow: don't increase the preferred size for empty windows if there is a size request set.

This fixes a reftest broken by commit 84e99b20ac806ee5f

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

x11: Avoid X error traps within server grabs

gdk_x11_device_xi2_window_at_position() may attempt to push/pop
a few error trap pairs while traversing the window tree. Move it
outside the server grab, and around the multiple XIQueryPointer
calls we may do here.

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

broadway: Fix memleak in error case

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

Add gtk_widget_set_font_options and gtk_widget_get_font_options

This allows a widget to override global font_options, such as hinting and
subpixel order. The widget's PangoContext is updated when this is set.

Some update code from gtk_widget_update_pango_context was moved to
update_pango_context so that gtk_widget_update_pango_context runs it.

http://bugzilla.gnome.org/show_bug.cgi?id=751677

switch: add a style property for the switch height

Instead of hardcoding an aspect ratio.
This will be replaced by min-height/max-height when we have them.

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

Updated Slovak translation

Updated Portuguese translation

Updated Portuguese translation

window: Don't send startup notifications for offscreen / popup windows

The WM isn't aware of O-R (popup) or offscreen windows. If somebody
maps an offscreen or a popup GTK+ window before the main window, we'll
complete the sequence before a "real" window is mapped. Make sure to
ignore these for startup notifies.

Updated POTFILES.skip

settings: Include pangofc-fontmap.h header on Wayland

In case the X11 backend is not enabled, we still need to include the
pangofc-fontmap.h header file, as we use the Pango/FontConfig API in
both the X11 and Wayland case.

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

wayland: Return the right type atom on gdk_selection_property_get()

We mistakenly forced the "STRING" type, which was able to confuse higher
layer helpers like gtk_selection_data_get_uris(). This fixes a crash
happening anytime a drop is attempted on a GtkPlacesSidebar.

wayland: Prepare against NULL selection owners on data_source requests

Currently, due to the lack of progress information in the Wayland DnD
protocol, we assume a DnD operation is finished after the first
data_source.send is finished (It's either that or leaving stuck grabs).

This however breaks previous assumptions that dest widgets can request
the data multiple times, even in response to GtkWidget::drag-motion.
This leaves us with a NULL owner for the DnD atom when we aren't
finished receiving wl_data_source events yet, causing a crash.

This commit fixes the crash, the behavior left is still far from
desirable though...

gtk3-demo: Group pango demos together

gtk3-demo: Add a fancy text demo

This demonstrates rendering text with a pattern instead of a
single color.

gtk-demo: Make the info bar example wrap

gtk-demo: Cosmetic changes

Clean up the code of many examples in minor ways, fix some memory
leaks, and avoid the use of dialogs where a regular toplevel works
just as well.

gtk-demo: Add a spin button example

More or less copied from the spin button example in testgtk.

cssvalue: Change color transitions to run premultiplied

Previously, the unpremultiplied values from the GdkRGBA were taken. Now
we premultiply the color values as specified by the CSS specs.
This is only relevant when transitioning with translucent colors.

An example is the halfway transition between transparent (0, 0, 0, 0)
and white (1, 1, 1, 1). Previously, all 4 values where transitioned
separately and the result was semi-transparent gray (0.5, 0.5, 0.5,
0.5).
By depending on the alpha value, the result is now semi-transparent
white (1, 1, 1, 0.5) which is what one would naively expect.

New reftest: color-transition

cssvalue: Special case resulting color for transparency

Again, preparation for using premultiplied color transitions.

cssvalue: Restructure RGBA transitions

Split out a function to make the code clearer for the changes ahead.

wayland: Set the master device on the drop-side GdkDragContext

And force the ungrab on it, instead of the slave, in the case of
local DnD drop. This avoids confusions on the pointer events spawn
from DnD, as GDK doesn't think anymore those are from a slave device.
Most namely, it fixes the stuck grab when finishing DnD on the
same app it was started from.

wayland: Split handling of clipboard/DnD data offers

We currently only hold the last offer received, which is wrong, as both
are independent and have different life cycles.

This means we have to store per-selection wl_data_offer and targets, and
maintain these as appropriate from the clipboard/DnD specific entry points.

broadway: Handle shm_open failing by falling back on tmp files

This is required if /dev/shm is not mounted on your system, which can
happen for instance in certain container configurations.

Adwaita: entry linking fixes

Set the right border color on the linked entry after the focused
one.

Adwaita: rubberbands!

Darken the rubberband a bit to make it visible on the selected
color background, fix treeview rubberband border issue.

wayland: Fix compile warning in DnD/selection code

Introduced in my recent commits

gtkdnd: Plug a leak

We create a cairo_t in order to find out the surface extents, but it
was being leaked, dragging the whole drag surface with it through
extra refs.

gtkmodelbutton: Plug a leak

The private struct had some data that needed freeing, but this widget
didn't have any destroy/dispose/finalize handlers.

gtkmenusectionbox: plug a leak

The list returned by gtk_container_get_children() wasn't being freed.

wayland: Plug some memleaks in DnD/clipboard code

gdk_atom_name() return value must be freed, this code was consistently
not doing so...

builder-tool: Don't strip h/vexpand

Setting these properties has the side-effect of setting the
corresponding -set properties, which makes GTK+ behave subtly
different. So don't mess with these.

GtkSearchBar: Don't vexpand

It turns out that setting vexpand to its default value has
side-effects that we need.

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

GtkImage: Make resource property work properly

gtk_image_set_from_resource was setting the resource_path,
only to have it overwritten again before returning. That
is not as it should be, so change things to set the resource
path late.

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

wayland: protect against unknown mimetype requests

This oddly can be reproduced with weston+weston-dnd, when dragging
anything from GTK+ into weston-dnd, it will insist on picking its
custom application/x-wayland-dnd-flower mimetype, and this request
forwarded by the compositor, even if GTK+ didn't announce it on
its wl_data_source mimetype list. (What should probably happen here
is that the request is silenced, and/or weston-dnd picks (null))

This should be harmless, we are leaking though the fd in that case,
because the emission of GdkEventSelection on an unhandled mimetype
results in NOP. In order to avoid this, we should check whether the
mimetype is supported at all on the backend code and possibly close
the fd, this involves storing these in the first place.

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

wayland: Ensure we close the fd on all error paths in data_source.send

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

wayland: close() the selection fd if we didn't start writing yet

If the other peer requests data too fast (too rare/unlikely though),
we might receive multiple gdk_wayland_selection_request_target() calls
with no ending gdk_wayland_selection_check_write(), in which case the
fd is leaked as no GOutputStream was created to take over it.

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

wayland: Ensure we cancel previous selection writes before starting one

We weren't catching all the places where the AsyncWriteData operation
should be cancelled, which could happen if we repeatedly request the
same target on different fds.

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

wayland: Do not close the descriptor on async_write_data_free()

At the moment we create the AsyncWriteData, the ownership of the
fd is granted to the GOutputStream, and the fd set to -1, so at
this moment we're just silently getting EBADFD.

This partially reverts 25885ca600fff, the initialization of .fd
to -1 is valid and stays though.

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

Updated Hungarian translation

wayland: Ensure device grabs generate crossing/focus events

On X11 this is something the windowing system does for us, which the
wayland backend should emulate, being grabs completely client-side.

So, if the grab and current focus windows differ, make sure we emit
focus/crossing events as it corresponds to the grab device.

wayland: Update grab serial when ungrabbing keyboards

This was being done so only on pointers. Internally, a GdkDeviceGrabInfo
is kept for each of the master pointer/keyboard, failing to do this for
keyboards results in a stuck keyboard grab.

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

wayland: Properly initialize/finalize where we store the data_source.send fd

The fd must be closed on async_write_data_free(), but we should also
initialize it to -1 so gdk_wayland_selection_check_write() doesn't wrongly
pick the stdin fd.

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

GtkWindow: some min/nat size corrections.

Don't add the container border to the title request size; it
is only used for the child widget.

Don't call gtk_widget_get_preferred_width_for_height() for
the title bar with an unrelated height and subtract the title
bar height before querying the child widget width.

Guard against negative size requests after substracting the
borders/shadows and the title bar.

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

3.17.4

Fix up doc comments

Fix up expected results of a11y tests

These changed due to th extra box that GtkAssistant now
inserts for each page.

Fix an oversight

GtkPlacesSidebar: Simplify sidebar code

Make this code follow GTK+ styles and conventions better.

GktPlacesSidebar: Plug a ton of leaks

If you don't keep in mind that g_object_get() returns references to
objects and copies fo strings, things leak all over the place.

sidebarrow: Avoid object-valued properties

They tend to leak...as these do.

render: Fix ridge/groove border rendering

The typo strikes again!

GtkHeaderBar: reduce minimum title width

The minimum title width affects the minimum window width
for CSD windows. To allow smaller windows like without
CSD reduce it a bit (276px vs 156px min width)

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