file chooser: Refine search progress display

Move the spinner to the search bar, so we can keep it around
until the search is over.

file chooser: Be more careful with subtitles

We were showing things like "Searching in (null)" if the current
folder is not in the sidebar. Avoid that by falling back to using
current_folder.

Pointed out by Carlos Soriano.

GtkApplication: avoid using stale timestamps

Avoid using a stale timestamp (from the last user interaction with the
application) when a message arrives from D-Bus requesting that a new
window be created.

In this case the most-correct thing that we can do is to use no
timestamp at all.

We modify gdk_x11_display_set_startup_notification_id() to allow a NULL
value to mean "reset everything" and then call this function
unconditionally on receipt of D-Bus activation requests.  The result
will be that a missing desktop-startup-id in the platform-data struct
will reset the timestamp.

Under their default configuration metacity and mutter will both map
windows presented with no timestamp in the foreground.  This could
result in false-positive, but there is very little we can do about that
without the original timestamp from the user event.

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

gtkframe: Don't queue a redraw on the entire widget every size allocate

Lots of applications often use GtkFrame as a giant toplevel container,
and that means that they get size allocated often. When frames get size
allocated, they invalidate their entire widget tree, even if they
haven't changed size or anything like that happens at all. This is
because the shadow / label needs to be redrawn if the child changes
size. We can optimize this out and only mark ourselves for a redraw if
the child has actually changed its size.

GtkButtonBox: remove spacing when buttons are linked

GtkButtonBox adds the "linked" class to its style context when its
layout is set to GTK_BUTTONBOX_EXPAND. It shouldn't ever make sense to
have spacing between buttons in that case, as themes generally draw
linked elements with a continuous border.

Thus, always set spacing to 0 and ignore GtkDialog's button-spacing
style property when the layout is set to EXPAND.

Also remove the now-redundant css rules which set button-spacing to 0
for message dialogs.

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

range: Tweak button bindings

Change things around so that warp-to-click and jump-by-pages are
bound to left-click and shift-left-click, depending on the value
of gtk-primary-button-warps-slider. Autoscrolling is bound to
right-click.

To achieve this, reorganize gtk_range_multipress_gesture_pressed
so that the functions are clearly separated.

Updated Spanish translation

Updated Spanish translation

GtkSettings: Fix a documentation typo

range: Trivial field ordering

range: Add a stepper alternative

Make shift right-click on the trough start autoscrolling. The
autoscrolling is similar to the way steppers operate now, with
the difference that we vary the scrolling speed based on the
distance of the pointer from the widget.

range: switch secondary and middle button on steppers

Clicking on steppers does different things depending on which
button you use. We used to scroll to the end on secondary click
and use high-speed autoscrolling on middle-click. Switch these
two around, since the scroll-to-end functionality is less likely
to be useful, and the seconary button makes the autoscrolling
more easily available.

font chooser: Remove a stray notify

This was forgotten when I changed to font map to not be a
property.

file chooser: Clean up unused includes

file chooser: Remove some obsoltete code

We no longer have editable rows in the list, so no need to
check for them anymore.

file chooser: Fix multi-selection

Even if we only ever hit the code with a singleton selection,
calling gtk_tree_selection_get_selected is not ok if the tree
selection mode allows multi-selection. Replace all calls to
gtk_tree_selection_get_selected in the file chooser code with
callback loops iterating over the selection. This problem
was introduced with the recently added rename and delete
menuitems.

file chooser: Do less work

We only need to update the sensitivity of the context menu
items rihgt before we are going to pop it up. Everything else
is pointless work.

file chooser: Avoid doing excess work

Whenever we change directories, we unset the model, and then
we set a new model. This causes several emissions of
GtkTreeSelection::changed, for each of which we do a bunch
of work to update the path bar, the location entry, etc.

We can savely ignore some of these signals, and do less work.

Trivial code cleanup

file chooser: Avoid animations when setting up

There is no need to animate things when we are just setting up
the startup mode, so disable transitions in the revealer and
the stack. Pointed out by Carlos Soriano

file chooser: Ensure we update subtitles

Notify the ::subtitle property when modes change that may
affect it. Also ensure that we update the location mode when
we switch back to the pathbar.

cssnode: Only queue_validate() when node is visible

This avoids running animations in hidden toplevels.

cssnode: Only invalidate sibling nodes if changed node is visible

Updated Portuguese translation

Added an additional explanation and code snippet for GtkImageMenuItem.c

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

Added a more meaningful description for /deprecated/gtkimagemenuitem.c

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

places sidebar: Fix rows getting 'lost' when dnd fails

There was some code that lead to the places sidebar loosing track
of the row being dragged if a motion event came in between letting
go of the button and the drag snap-back animation ending. This would
cause us to not show the row again, giving the appearance that it
was 'lost'.

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

testtoolbar: Don't expand rows that shouldn't be expanded

frame: Remove get_path_for_child implementation

As far as I can read the code it added the .frame style class to the
GtkFrame element. But GtKFrame already has a .frame style class these
days.

actionbar: Remove get_path_for_child implementation

It wasn't used because CSS walks the widget->parent chain, so it'd only
ever call this func for priv->revealer.

css: text-decoration-style is not inherited

Simplify Xft setting fallback

If we don't find Xft values in the X resource db, simply fall
back to the values that are hardcoded in /etc/X11/Xresources
anyway. Extra trickery with likely-made-up screen dimensions
is not going to yield better results, and only makes for a
deeper rabbit hole when debugging.

gtksidebarrow: remove unused property

The sensitive property was a workaround which commit
3f8982a0cd7b779fe023bbd fixed.
Just remove it now that is not necessary.

gtkrevealer: fix preferred width calculation

A typo which was using minimum_width for natural_width and viceversa.

Fix a minor typo

Updated Polish translation

gdkwindow-quartz: partial aspect ratio support

Support was added for GDK_HINT_ASPECT in
gdk_quartz_window_set_geometry_hints though with one restriction:
min_aspect and max_aspect have to be equal, which I believe corresponds
to the most common usage. A warning will be printed if this condition is
not met but min_aspect will be used anyway.

testgmenu: use the menu binding code

The menu tracker does a better job of this than we can, so move over to
using it instead.

This fixes issues with './testgmenu --import' not properly displaying
the language submenu.

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

gtkdnd: Fix introspection annotations in some functions

The GdkEvent field is nullable on gtk_drag_begin*, and gtk_drag_cancel()
is a method, like the rest of the DnD API.

gtkdnd: Account for setting a same icon helper

g_set_object() will take care of ref'ing before destroying the previous
instance, which might actually be the same pointer.

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

Updated Portuguese translation

file chooser: Add a shortcut to focus the sidebar

Alt+P used to do this in the gtk2 filechooser (well, sometimes).
Add this back as a key binding.

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

place sidebar: Make removing bookmarks work again

This was accidentally broken in ae8c07c67cfa83d967 when I
forgot to switch the sense of a condition.

file chooser: Make delete confirmation match nautilus

No need to diverge here.

file chooser: Separate out delete and trash

We only ever show one of the two context menu items (and we prefer
Move to Trash over Delete). Only use the confirmation dialog when
deleting.

place sidebar: minor change

We generally prefer for loops to iterate over lists.

file chooser: Allow rename-to-self

When checking for a name clash for renaming, don't complain if the
name is unchanged. That's harmless.

wip: reshuffle name checks

file chooser: Remove unused field

file chooser: Clear search entry when it goes away

Keeping the entry contents conflicts with type-to-search.

css: support text-decoration-style

The support is limited to underline single, double and wavy, which
is what pango has today.

wayland: print correct coordinates during DnD

Convert wl_fixed values to floats for printing, rather than printing
them as non-meaningful integers.

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

gtkdnd: Traverse across insensitive widgets

The current widget lookup code bails out on insensitive widgets, there's
however legit cases where we want DnD handled by a parent of the insensitive
widget, so just keep going upwards in that case.

We also use now the widget state flags, because get_sensitive() doesn't
propagate across hierarchies, so we could conceivably find a drop site
inside an insensitive widget.

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

gtkwindow: Only allow unrestricted positioning to text handle popovers

This behavior has been made optional on add_popover() time, text handles
will keep being able to overflow the window, in order to allow text
selection on views too close to the window edge.

Regular GtkPopovers are reinstaurated to the previous size positioning
logic though, that is, limited by the visible area of the window.

scrolledwindow: Keep scrollbars out of GtkScrollable::get_border

It looks a bit odd that scrollbars stay over treeview headers and
similar, seems nicer to just avoid the border regions.

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

cssmatcher: Make nth-child(odd) work on first line

.. instead of only on 3rd, 5th, 7th, ...

cssnode: Do finer-grained position invalidation

As a side effect, this should fix this bug:

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

csstypes: Change NTH_CHILD and NTH_LAST_CHILD change propagation

For now assume that if those flags change for a node, they also changed
for all sibling nodes. This is not strictly true but simplifies
invalidation.

The reason it simplifies invalidation is that when removing or adding a
node, we can just invalidate the first node (for NTH_LAST_CHILD) and the
new node (for NTH_CHILD) and all the other nodes will be invalidated
automatically.

cssselector: Report finer granularity for position changes

csstypes: expand GtkCssChange enum

Instead of GTK_CSS_CHANGE_POSITION we now have 4 values:
GTK_CSS_CHANGE_FIRST_CHILD, GTK_CSS_CHANGE_LAST_CHILD,
GTK_CSS_CHANGE_NTH_CHILD and GTK_CSS_CHANGE_NTH_LAST_CHILD

Nobody is using them directly yet.

gdkdisplay-x11: The leader window should be an input-only window

This prevents it from creating a _NET_WM_USER_TIME_WINDOW, which can
confuse mutter / other window managers, and also me, when debugging.

pango: make merge_attrs return the merged list

This simplifies all the callers

css: add shorthand text-decoration property

Note that for now we take into account only text-decoration-line
and text-decoration-color

css: support text-decoration-color

linkbutton: use the text-decoration-line css

Remove the custom add() implementation and use css to underline the
label.

css: add support for text-decoration-line

entry: add css letter-spacing

label: add support for CSS letter-spacing property

Support letter-spacing CSS property on GtkLabel.
Reftest is included.

pango: move an utility function from gtklabel

This will be used both by gtklabel and gtkentry

Updated Greek translation

Fix return value error in _gtk_file_system_model_get_directory

Updated Portuguese translation

file chooser: Allow deleting files

This is another often requestsed feature for save mode.

Based on a patch by John Beard,
https://bugzilla.gnome.org/show_bug.cgi?id=325150

file chooser: Allow renaming files

This has often been requested as a useful feature in save mode.

Based on a patch by John Beard,
https://bugzilla.gnome.org/show_bug.cgi?id=325150

Fix up some whitespace mishap

file chooser: Redo the trailing space warning

Redo this slightly differently, so we can keep all the simple
checks in one place. This will make it easier to reuse the code
for file renaming.

places sidebar: Fix memory handling of rename popover

We were freeing the same string twice here, leading to
badness.

Strip leading and trailing whitespace from filechooser save filenames

This makes a local copy of the file part of the entry to strip it transparently
Since this is assumed to be a mistake, the user is not notified.

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

Warn for whitespace at beginning or end of new folder names

The warning is not intended to disable the Create button and must only be shown
when the folder is not found, so this is implemented in the folder name exists
callback.

A "name" entry was added to FileExistsData to pass the filename to the callback

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

docs: Drop deprecated symbolic icon names

Do not suggest using icons that do not exist.

container: Update the add() warning

Now that gtk_widget_reparent() has been deprecated, we should not
suggest it.

Updated Portuguese translation

Updated Portuguese translation

file chooser: Make Ctrl-L work in search mode

There is no real reason to not allow this. The transition is
not quite as smooth as it should be, currently, but at least
it works.

file chooser dialog: Make button not take focus

Things work smoother, keynav-wise, if the search button does not
take focus on click.

file chooser: Make unsetting save entry work

Fix an overzealous assertion.

file chooser dialog: Make action changes work

We were not listening for action changes to update the
save entry.

file chooser: Cleanups

testfilechooser: Add a --local-only flag

This allows for easier testing of !local-only mode.

file chooser: Don't show Trash in save mode

You can't save there...

GtkPlacesSidebar: Allow hiding Trash

We will use this in the filechooser in save mode.

GtkTreeViewColumn: Fix a documentation typo

file chooser: Improve column sizing

Allow the name and location columns to be resized, but arrange
for their sizing to be reset when the column layout changes (either
by the location column appearing/disappearing, or by the time
column changing between mtime and atime. This gives a decent
compromise between good automatic sizing and user control.

file chooser: Remove a wrong assumption

The code for getting the selected files was assuming that
we are always in browse mode, and was causing warnings when
hitting Ctrl-L twice, right after opening the file chooser.

The fix is to simple use the model that is passed into the
callback.

file chooser: Don't show Enter Location in sidebar

Following the file chooser mockups.

file chooser: Update empty state for search

Follow the mockups more.

file chooser: Add search progress

Show a spinner if no search reasults have shown up yet and
the search is still going, after 2 seconds.

file chooser: Set fixed sort functions for recent and search

file chooser: Improve location column formatting

file chooser: Improve the mode switching operation

Avoid reloading models unnecessarily, e.g. when the user
switches to search and back without starting a search.
Keep the current list contents visible until a search is
actually started. Also, synchronize any changes in the
column layout with the corresponding model changes.