Due to made widget changes in fill_in_all() the focus can change
from the full name entry elsewhere. Remembering focused widget and
restoring it at the end of the function makes the editor being
opened with full name entry instead, as expected.
The last time we needed to explicitly depend on a particular g-i-t
version was when "default-avatar" was added, but that was nearly 4
years ago.
The new "adwaita-icon-theme" dependency breaks the build on anything
less than a bleeding-edge environment, so I think we'd be better off
just leaving the dependency implicit.
Alternatively we could still explicitly check for gnome-icon-theme as
a fallback for adwaita-icon-theme, but meh... it's not worth it IMO.
In some cases, like on errors or cancel, there could be left
an information in the message-list that the list is generating
its content, while it didn't do it in reality. This always resets
the message-list information in case the current message-list
generation finished and was the last in the queue.
Since GLib begun to claim about incorrect ids being passed to
g_source_remove(), a problem with EActivityBar had been discovered,
it doesn't unset its priv->timeout_id when the timeout is reached,
which leads to the new runtime warnings from GLib.
It used to copy the link into both buffers, same as the 'Copy Email Address"
does, but it seems, after a move to WebKit based message preview, this
changed and the link had been copied only to the selection clipboard.
Make the function consistent and copy the link into to both clipboards.
Otherwise invalid source ID is used in the call of g_source_remove()
when is tried to remove "pending" auto-mark-as-seen timeout callback.
It can show a runtime warning on a console too.
This used to be done in mail_cancel_all(), but since commit 8ae3cc3283
the function is gone and the mail operations are not cancelled, which
prevents the application to go offline or to quit in a timely manner,
because it's waiting for a finish of possibly expensive operations.
