gtkrange: Instaurate GTK+ grabs while manipulating ranges

It should not be necessary for most situations, except while there's
GDK grabs coercing events in a different way.

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

gtkgesture: Do not ignore events from other GdkWindows than the widget's

There's GDK grab situations (eg. pointer motion outside the grab window
in combination with a GTK+ grab) where a gesture may receive events from
windows that are not the widget's.

The _update_widget_coordinates() still does work for those situations, so
just let these events go through instead of ignoring them.

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

gtklabel: Fix touch link handling under wayland

Refactor the code updating the active link under the current coordinates
into a separate function, and call it on GtkGestureMultiPress::pressed
so the link is updated on GDK_TOUCH_BEGIN. Based on a patch by
Jan-Michael Brummer <jan.brummer@tabos.org>.

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

gtklinkbutton: Set cursor on button's event window

Setting the cursor on the widget window (i.e. the parent widget's) is
finicky because the cursor needs to be updated on crossing events, and
will yield the wrong result for other master devices that happen to be
in other areas of the same parent widget's window.

Just set it always on the event window created by the GtkButton parent
class. That window was causing the crossing events, so the rectangle
that gets the hand cursor set will be the same size, and we don't need
to track pointer crossing state that way.

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

wayland: Clear tablet tool cursor on proximity out

This ensures that the tablet tool will get the cursor updated even if it
happens to fall within the same window again on the next proximity in.

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

gdkwindow: Update cursor for tablet tools in the window

This check must be done explicitly on Wayland as the master device for
tablet tools differ from the Core Pointer. This ensures that whenever a
tablet tool is inside a window and the cursor is programmatically changed,
it will be visually updated too.

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

wayland: Observe GDK_SEAT_CAPABILITY_TABLET_STYLUS on gdk_seat_get_slaves()

This flag wasn't being honored so far...

wayland: Implement support for tablet wheel scrolling

Adds support for creating scroll events from Wayland tablet wheel events.
Even though no Wacom tablet puck has a smooth-scrolling wheel, both event
types need to be generated to make the upper layers happy.

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

wayland: Make function to create scroll event more generic

Add GdkWaylandPointerData and GdkDevice arguments so it can be
used across master devices.

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

wayland: Get implicit grab serial information from tablet devices

If a tablet device is used to perform actions like window moving or resizing,
GTK must provide the correct implicit grab serial number over Wayland to Mutter
in order for the action to succeed. This commit adds tablet support to the
implicit serial getters.

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

wayland: Throttle system bell requests

If a bad behaving application tries to make the window/display beep too
often, throttle the beep requests so that we don't end up filling the
Wayland socket queue.

The throttle is set to 50 beeps per second, which far more beeps than
will ever make any sense from a user experience point of view, but will
avoid terminating due to an excessive amount of requests.

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

wayland: Make beep requests go through the GdkDisplay

This way we can add things like throttling.

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

3.22.17

update expected test results

filechooserwidget: Let the rename entry hexpand

This makes a difference with other locals where error messages are
longer than the standard width of entry+spacing+button.

Add nullable return annotation to gtk_text_mark_get_name

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

Add nullable return annotation to gtk_notebook_get_tab_label

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

Update API for El Capitan+

Use NSAnimationContext where possible.

os x: Add new versions

Update the URLs of known licenses to HTTPS

Use HTTPS by default for security/privacy. Avoids slow redirects.

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

gtkapplication: Mark gtk_application_get_active_window() as nullable

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

Copy gtk_clipboard_get_default implementation to gtkclipboard-quartz.c

Fixes bug 784323.

Update POTFILES.in

GtkFileChooserNativeWin32: add support for get_filter and set_filter

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

testgtk.c: native file chooser filter changes

Set the filter before showing the dialog
Afterwards, fetch the last active filter and display its name.

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

GtkFileChooserNativeQuartz: add support for get_filter and set_filter

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

GtkFileChooserNative: documentation updated

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

GtkFileChooserNativeQuartz: add support for filters

Includes:
* Simple glob patterns (*.ext, *.*,...)
* MIME types
* pixbuf formats

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

GtkFileChooserNativeQuartz: add partial support for extra widget

When the extra widget is a GtkLabel, then its text will be displayed as
a message in the NSSavePanel or NSOpenPanel

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

GtkFileChooserNativeQuartz: add support for GTK_FILE_CHOOSER_ACTION_CREATE_FOLDER

Which is via NSOpenPanel, not NSSavePanel...

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

GtkFilechooserNative: add macOS support

Based on the Win32 implementation, as well as the macOS file chooser
from
https://github.com/GNOME/gedit/blob/master/gedit/gedit-file-chooser-dialog-osx.[ch]
Not fully tested yet, but working properly so far.
TODO: filter support, extra widget (label), documentation...

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

Add Friulian translation

image: Fix loading of pixdata GResources

Pixdata is deprecated but some software already use GtkImage widgets
with image data loaded from GResource-backed pixdata. As the
security-problem ridden pixdata loader was removed, we need to manually
check whether the GResource data is pixdata, and load it manually.

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

Avoid a race in the print portal

Same as the previous commit; use the new predictable request
object path to connect to the Response signal early.

Avoid a race in the file chooser portal

Use the new predictable request object path and connect
to the Response signal before issuing the portal call.
This avoids a race that is pretty unlikely to hit in
the filechooser case.

label: Remove extra quote in code example

GdkRGBA: Fix typo precentage => percentage

x11: Assign GDK_SOURCE_TABLET_PAD when necessary

Wacom tablets often have a "pad" device which houses multiple buttons. At
present, these devices are incorrectly marked as GDK_SOURCE_PEN which can
cause problems for some software.

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

migrating-StyleContext: CSS does support bindings

so remove the bit in the migration guide that says it doesn’t.

css-overview: Fix typo “{with=>which} is disabled”

Update Kazakh translation

GdkWaylandWindow: Fix version numbers in export_handle() docs

This change appeared first in 3.22.16 release, not 3.22.13.

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

3.22.16

Update a11y tests

Expected output of a11y tests has changed.

Updated Czech translation

Updated Spanish translation

gdk: Set vid/pid on wayland tablets

This was missed so far... Use %.4x format, in order to behave just
the same than X11.

headerbar: fix "widow" typo in docs

Updated Norwegian bokmål translation

GtkActionHelper: Remove unnecessary NULL check

This NULL check has existed since commit 652f16dd985dbc1b, when
GtkActionHelper was first introduced, but so has the assertion at the
top to ensure the validity of 'widget'.

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

GtkActionHelper: Allow a NULL action-name to unset the previous GAction

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

file chooser: Don't pass NULL as title to the portal

This causes critical warnings.

printunixdialog: Update collate icon as entry changes

By relying on GtkSpinButton default activation behavior, the
collate icon doesn't get updated when a new number is typed
in the copies spin button.

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

gtktextdisplay: remove unused code

In render_para() function, which is called for every text line that
needs to be drawn.

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

dist icon.list

Otherwise things will not work in a release tarball.

icon-browser: Add a few icons

These have appeared recently.

icon browser: Make icons a bit larger

Use 48x48 in the list, instead of 32x32. We have the space.

icon browser: Add a 'copy to clipboard' button

This makes it more obvious how to use the icon.

icon browser: put icons as data

Instead of hardcoding all the add_icon calls,
read the icon list from a keyfile, for easier
extensibility.

testsuite/scrolledwindow: Try non-overlay/non-auto

It was only testing the default configuration, where overlay scrolling
is on and both scrollbars use POLICY_AUTOMATIC. We should also test the
other 3 configurations that are available by including non-overlay
scrollbars and/or those that use POLICY_ALWAYS.

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

ScrolledWindow: Don’t req size for autohidden bars

POLICY_AUTOMATIC means scrollbars are only shown when needed, i.e. when
the size of the window is not large enough to show the entire child. So
when measuring the preferred size, such scrollbars should be ignored.

But measure() added size for *any* non-overlay scrollbar of the opposite
orientation, e.g. for horizontal size, it added the width of vscrollbar.
So we requested for child + bar, & having enough for child meant that the
policy hid the bar, leaving extra space empty below/right of the child.

Fix this by only adding size for such bars if they use POLICY_ALWAYS.

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

scrolledwindow: Avoid critical when removing source

Make sure indicator->conceil_timer is actually set before trying to
remove it.

wayland: selectively cancel key repeat on key release

Under Wayland, when multiple keys are pressed and the user releases a
key, key repeat should continue unless the key released is the one
currently repeating.

In the case of:

- key1 press
- key1 repeat
- key2 press -> key1 repeat stopped
- key2 repeat
- key2 release

The behavior should be to cancel keyboard repeat, though key1 is still
held down. This is consistent with prior X11/XWayland behavior.

The following also must work:

- key1 press
- key2 press
- key2 release
- key2 press
- key1 release
- key2 should continue to repeat

The fix for bug #778019 should continue to work:

- key1 press
- key1 repeat
- key2 press -> key1 repeat stopped
- key1 release
- key2 should repeat

The choice to change the counter nkeys to the flag repeat_active
helps to solve the second test case.

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

wayland: fix xdg_surface test in move/resize drag

begin_resize_drag() and begin_move_drag() check for xdg_surface being
not null, but those apply on xdg_toplevel so they should check for
xdg_toplevel being non-null instead.

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

wayland: check parent surface for subsurface

When an event is received while a tooltip is showing, the GtkTooltip's
event handling code can end up calling gdk_window_set_transient_for()
from gtk_tooltip_set_last_window().

The Wayland GDK backend will try to automatically create a subsurface
in gdk_wayland_window_set_transient_for() but if the parent surface is
gone meanwhile, this will will cause a crash when trying to create a
subsurface from a parent with a null surface.

Checking for the parent is not sufficient, we ought to check for the
parent surface as well to avoid the crash.

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

wayland: Do not map toplevel utility as popup

Applications can specify the type hint as utility even on toplevel
windows.

When that toplevel is also marked as a transient for another window,
GDK Wayland backend would translate that as an xdg_popup which is not
appropriate.

While utility temp windows should remain mapped as subsurfaces (such as
the ones used by treeviews), regular windows should not translate as
neither a subsurface nor an xdg_popup.

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

gtkfilechoosernativewin32: Fix support for non-ASCII paths

The code used SIGDN_URL to get an URL for the selected item, but Windows URLs
are a mix of unicode and percent encoded characters in the locale encoding
and not something GFile can understand. The result is a garbage file
path.

Instead use SIGDN_FILESYSPATH to get a real file path if available.

Also checks the return value of g_utf16_to_utf8 because file paths on
Windows can contain lone surrogates which would make the conversion fail.

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

Redo the listbox dnd example

Rewrite this example to have better drag highlighting.

Adwaita: Add missing rule for fullscreen headerbar

Another selector forces round corners for headerbars in a stack, and it
has higher priority than the selector covering the non-stack case from
commit 712a8adbd9a11fcebce0269562e9a028a17fa862. Totem’s MainToolbar
happens to be in a stack, and we should maintain symmetry here anyway.

So, as window classes .maximized and .tiled are excluded from this other
selector, the newly handled .fullscreen case must be excluded here also.

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

HighContrast: Regenerate CSS files with sassc

HighContrast: Update parse-sass.sh; remove Gemfile

Make parse-sass.sh use sassc instead of the old Ruby method.

Adwaita: Remove unused Gemfile

themes: Square corners on fullscreen win headerbar

Totem uses a fullscreen window with a headerbar at the top, and without
this change, that headerbar has rounded corners, which look different
from a maximised window and let video content show through beneath.

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

Updated Lithuanian translation

wayland: Don't warn if the display is lost, skip exit handlers

There is no need to have every application log a warning when the
Wayland display server goes away, and we are using _exit instead of
exit elsewhere.

This is also what the X11 backend does (see gdk_x_io_error).

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

Update Serbian translation

wayland: Don't abort when preparing the source if connection is lost

Aborting the application makes it look like an application bug, when
it is the expected thing to do when the Wayland display server goes
way. eg., when the user logs out. The log level is also demoted to
avoid a storm of warnings in the log from all applications whenever
this happens.

This is also what the X11 backend does (see gdk_x_io_error).

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

textview: Make GTK_TEXT_WINDOW_PRIVATE public

The GTK_TEXT_WINDOW_PRIVATE enumeration value is really *not* private.
Internally, it's used as a simple "invalid value" marker, and
application and library developers are supposed to use it as such in
their own code.

Let's just document it, and since the GtkTextView documentation and
internals go a long way to state the fact that it should not be used as
an argument value, let's add some pre-condition checks as well.

This commit fixes GtkSourceView's use of GTK_TEXT_WINDOW_PRIVATE as
default value for a GObject property that was broken by the change in
glib-mkenums to honor the `/*< public >*/` and `/*< private >*/`
trigraphs.

tests: Add testforeign

Add a test for exporting a handle. There are no GTK+ API for this, but
only per backend GDK API, and so far only Wayland is supported. There
is a private GdkWindow API but it's not exposed externally.

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

GdkWaylandWindow: Allow calling export() multiple time

Allow getting the same export handle multiple times by calling
gdk_wayland_window_export_handle() multiple times. For each time
export() is called, a unexport() must be called to unexport.

When the window is already exported, the exported callback is called
via a idle handler. If there are multiple export() calls, they are
invoked in order either when the handle is received by the display
server, or when the idle callback is invoked.

Calling unexport() will not affect future invokations of the exported
callback, unless all export() calls have their unexport() call count
matched.

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

Move style change for popover creation.

Creating with `gtk_popover_new_from_model` should be exactly the same as
if via `gtk_popover_new` plus `gtk_popover_bind_model`.

Also remove the style if the model is unbound at any point.

widget: Remove useless assignment

We don't use adjusted_allocation after this line.

examples/drawing: Don't call gtk_main_quit

This is a GtkApplication so it will do the right thing when closing the
window.

fallback-c89: Include config.h

It holds all the HAVE_<func> definitions.

widget: remove double assignment

We never read the value we assign here.

placesview: Remove duplicate GtkMountOperation assignment

we assign a new mount operation a few lines later again.

levelbar: Don't allocate 0 blocks

Could result in a division by zero later on and doesn't make a lot of
sense anyway.

grid: Remove unnecessary NULL checks

The minimum and natural pointers passed to measure are never NULL and
that's the only place where we call gtk_grid_get_size_for_size.

GtkGestureMultiPress: Remove superfluous NULL check

The rect parameter in gtk_gesture_multi_press_set_area is annotated as
nullable and the code handles the rect==NULL case, but the
g_return_if_fail kept that case from ever happening.

label: Remove dead ternary operator

filechooserwidget: Remove always-false error case

_gtk_search_engine_new never returns NULL, it always returns a valid
GObject with at least the simple search engine initialized.

sidebarrow: Remove useless size group

Nice try, but size groups don't work with invisible widgets anyway.
Invisible widgets request 0×0.

label: Remove useless if statements

We never pass nullable pointers into get_size_for_allocation.

gtkwindow: Hide CSD title bar when undecorated

When switching from decorated to undecorated, the title bar should be
hidden as well.

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

docs: Fix copy-paste error in gtk_file_chooser_get_extra_widget() doc comment

Adwaita: Avoid accindental border on last treeview header

The :last-child selector supposed to reset the border was
overridden by the :hover selector. This is fixed by moving the
:last-child selector after the overriding one.
Thanks to Sebastian Keller for spotting.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=779078.

Adwaita: regenerate the css with sassc

Adwaita: modify parse-sass.sh script to use sassc

EntryBuffer: Don't generate changed events when input is truncated

widget-factory: Use :relief=none, not class .flat

The rest of the ui file follows that convention.