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

GtkWindow: fix default empty window size with CSD

In the non-CSD case we checked for 0x0 window size requisition
and replaced it with 200x200 so the window was still visible.
This no longer works in case of CSD as the shadow and title bar
are always added to the requisition resulting in a titlebar/shadow
only window in case there is no child widget (this is currently
visible under wayland or when setting GTK_CSD=1).

Instead of special casing the final window size, special case
the child requisition paths instead. This gives us the same
requisition in both, CSD and non-CSD cases (the header bar
has a too large minimum width atm so the resulting window is
still not the same)

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

mir: fix toggling between maximized and restored

mir: fix wheel scrolling in both directions

mir: emit GDK_ENTER_NOTIFY on pointer enter

mir: remove non-useful logging

mir: don't need dummy egl surface

mir: emit configure event on window move/resize

Without this, some windows don't restart the paint clock needed to
actually render the contents.

mir: fix build due to missing api

Adwaita: scale marks visible on the dark variant

Updated Hungarian translation

GtkScale: More work on mark spacing

Try harder not to push the scale out of balance by marks. With
this change, the remaining difference in the drawing is down
to asymmetric assets and margins coming out of the theme.
To fully support such asymmetric rendering, we need implement
baseline alignment for scales.

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

printing: Get output bin via IPP

Request "output-bin-supported" and "output-bin-default" attributes through
IPP if there is no PPD for selected printer.
Pass "output-bin" option with other options in printer_get_options().
Translate standard IPP values of "output-bin" option.

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

GtkStack: Avoid some unnecessary complication

We already have an in-destruction flag, so use it.

Fix calculation of edge input window size in case the real window is too small.

This error resulted in warnings like
"pixman_region32_init_rect: Invalid rectangle passed"

In case the window is smaller than handle_size * 2 the resulting
edge window got a negative size. Prevent that by limiting the
handle size to half the respective edge length. This also
prevents the corner windows from overlapping in case the window
is too small.

Revert "window: Check if we can use CSD before enabling them"

This reverts commit c5e5ee67490e7e7af56052d8f8beb75db002c2f1.

It turns out that this commit breaks the default sizing of GtkWindow,
which in turn causes the window to be smaller than the handle size,
which ends up sending invalid rectangles to Cairo.

We'll have to find another way to fix the CSD issue.

See also: https://bugzilla.gnome.org/show_bug.cgi?id=751140

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

Avoid spurious operations on destroy

When the stack is destroyed we do not want to waste time running
animations and notifying listeners about which is our current
visible child.

This is not only an optimization, but it is important for the stack
switcher widgets: since they are in another branch of the hieratchy
we do not want to get notifications while the stack is being destroyed.

Based on a patch by Paolo Borelli
https://bugzilla.gnome.org/show_bug.cgi?id=724506

Pacify gobject-introspection

gobject-introspection complains about the parameter names of
vfuncs. Why, you ask ? No idea.

GtkRange: Improve zoom handling a bit

The calculation to update the initial slider position on zoom
changes was not working correctly when using keys to toggle
zoom on and off for scales. Avoid it by updating the position
beforehand.

GtkRange: Make drag gesture less agressive

Moving the mouse while pressing one of the steppers was making
the slider jump to the end, unexpectedly. This was caused by the
drag gesture kicking in when it shouldn't. Fix this by making
all drag gesture signal handlers only do something if we are in
a drag thats started on the slider.

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

file chooser: Avoid tall combos

The filter combo box in the file chooser was stretching to match
the height of the extra widgets next to it. Since, stretched combo
boxes are not a pretty sight, make it stick to the top instead.

Patch by Caolán McNamara,

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

GtkOverlay: Sync up parameter names

gtk-doc gets confused if the parameter names between header,
source and doc comment don't match up.

Fix a typo

gtk-doc complained about a multi-line Since: tag, but the
real culprit was a typo in the Deprecated: tag.

Tell gtk-doc to ignore private headers

Tell gtk-doc to ignore new private headers as well, otherwise
it complains about undocumented symbols in them.

GtkWindow: Small documentation addition

GtkTextView: Small documentation addition