Refactoring: that function should not be publicly accessible
Ubuntu: Lazy load Settings tabs

Only create the settings tabs when we need them

REVIEW: 117817
Prevent duplicate signals on model recreation

This is directly related to commit e9619f03311aa71f16bfad41e97011757e327727
which changed the lifetime of ImapAccess. Previously, it used to be recreated
whenever the IMAP Model got recreated, but that commit changed ImapAccess to be
created exactly once (and in future, once for every IMAP account).

The rest of the signals appear to be fine; they do not connect ImapAccess, but
instead work on various models which are published by ImapAccess. In future when
ImapAccess changes to emit signals when the models actually change (and not when
e.g. an unrelated piece of settings like the human readable name, or the SMTP
prefs or something else change), the reconnecting of signals should happen at
that level.

This double-emitting led to a bug when a new account was added the first time,
and some SSL errors were shown. By mistake, two dialogs would pop up, and if a
wrong password was entered, the Q_ASSERT on OpenConnectionTask.cpp, line 493 was
hit.
SVN_SILENT made messages (.desktop file)
SVN_SILENT made messages (.desktop file)
Ubuntu: Password dialog fix Cancel button not loading settings tabs

REVIEW: 117912
Ubuntu: Fix cmake to copy qml files pre-build

REVIEW: 117896
BUG: 333075
Ubuntu: Fix the ImapSettings flickable

This fixes the view was not scrolling when OSK is visible.

REVIEW: 117924
GUI: make sure the composer always has a window title

Simply opening the composer as a new fresh mail, without replying to an existing
subject, would leave the window title uninitialized, which means that the user
would get an ugly "trojita" instead of a nice, localized "Compose mail".
Ubuntu messageListPage delete message

BUG: 333212
REVIEW: 117514
Get rid of the duplicated FLagsOperation

The QML is rather hostile towards working with enums which come from the C++
code *and* are not already wrapped in a QObject-deriving class. Instead of
duplicating the existing FlagsOperation enum partially, it seems that it is
cleaner to give up on the API design principles which ask for using enums
instead of plain bools for readability, and use a simple bool in this case.
Merge branch 'ubuntu-delete-message'
No need for "explicit" in this ctor anymore
IMAP: relax quoted-string parsing to allow for Groupwise's escaped parentheses

As per the bugreport, Groupwise does not conform to RFC3501 and sends
parentheses ("(" and ")") as escaped characters within the quoted strings. The
RFC 3501 is pretty clear in only allowing escaped quotes and backslashes.

This change let's Trojita process these broken responses. It is a tough call to
tell what to do, but interpretting these by ignoring the backslashes is probably
the lesser of the two evils.

BUG: 334456
REVIEW: 118033
IMAP: Accept Exchange's botched body-fld-dsp

According to a report by Martin Steigerwald <ms@teamix.de>, Exchange sends a
body-fld-dsp which contains complete crap.

CCBUG: 334056
REVIEW: 118035