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.

Add a getter for the directory to file system model

This will allow us to aovid reloading if the directory is
unchanged.

file chooser: Clean up enum handling

Use g_settings_get_set_enum to handle the clock and date format
settings.

file chooser: Code cleanups

Shorten excessively long field names.

file chooser: Consistently clear model caches

This was not done for all models in all places, leading to e.g.
icon theme updates not applying correctly to search results.

file chooser: Add a way to toggle time display

file chooser: remove a stale comment

file chooser: Improve time and date formatting

Use the same formats as nautilus, and put date and time in
separate cells, so they are neatly aligned.

file chooser: Don't allow resorting recent files

They are inherently sorted by access time.

file chooser: Use access time for recent files

Use access time for sorting the recent files, and show it in
the list instead of mtime.

file chooser: Add and use a model search engine

This search engine reuses the GFileInfo that is already loaded
for the file list, to ensure that hits from the current directory
always appear promptly.

testfilechooser: Disable the terrible preview

It makes testing the intended user experience harder.

file chooser: Don't show recent in save mode

Don't show Recent in the sidebar when we are in save mode.
We also ignore the startup-mode = recent in save mode now - we
don't want to populate the file list with recent files if Recent
is not on the sidebar. If you really want to go there, you can
still enter recent:// in the location entry.

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

file chooser: Add Ctrl-F as search shortcut

This is what nautilus uses, and is more discoverable than the
somewhat easter-egg Alt-S, which is still supported.

file chooser: Make search shortcut toggle

This is more in line with having a toggle button for search,
and it matches what we do e.g. in nautilus.

file chooser: Tweak bindings in save mode

In save modes, the entry is really more for entering a name than
entering a full location, so don't go there eagerly when '.', '/'
or '~' are pressed. Make Ctrl-L work better in this mode too.

file chooser: Trigger location entry not just from file list

Use the same code that brings up the location entry on '/',
'~' or '.' also when the focus is not on the file list. This
prevents those key presses on the sidebar from ending up in
the search entry.

file chooser: Open location entry on '.'

Typing ./ is a fairly common practice to enter a location
relative to the cwd, so make this open the location entry.

Revert "file chooser: Open location entry more easily"

This reverts commit e6c1cbc88ebbf51dcf4ca2b4b0016c81808d7889.

Revert "file chooser: Don't start search too eagerly"

This reverts commit f126157bfa9e8a550067fa169de24697add028bb.

file chooser: Improve keynav

Some internal containers were erroneously taking focus, interrupting
the flow of tab keynav, and using arrow keynav to go from the file
list to the sidebar did not work anymore, after the recent sidebar
rewrite.

file chooser: Give the location column more space

Giving it 15 characters makes it at least fit "Documents" without
ellipsizing.

file chooser: Sort search results better

Set a default sort function that sorts results from the
current folder first.

file chooser entry: Capture Escape and emit :hide-entry

Make the file chooser entry optionally capture Escape
and emit a signal. Make the file chooser widget hide the
entry on that signal and go back to the path bar.
This gives us a two-level undo:
location entry -> path bar -> dialog close.
When the location entry is permanently displayed in the
header for save mode, we still let the first Escape close
the dialog.

file chooser: Make Escape work in search entry

Arrange things so that hitting Escape during a running
search stops the search, but leaves the search results
around, and hitting Escape again leaves the search mode.

file chooser: Make location column work better

The location column did not work for search results in recent://.
Fix that by looking at the target uri in this case. Show the location
column in recent mode. And make it more similar to nautilus by
showing the full path if it is not below $HOME.

file chooser: Make search in recent work

All that was needed here was to set the query location.