Fix the docs according to the current double-click option place (take two)

DolphinView: set the parent of layout in the ctor

This silences a runtime warning:
QLayout: Attempting to add QLayout "" to DolphinView "", which already
has a layout

Remove redudant setLayout() calls, passing a parent widget to the
Q*BoxLayout ctor sets that layout as the top-level layout for that widget.

Fix the docs according to the current double-click option place

GIT_SILENT Update Appstream for new release

(cherry picked from commit 9b8ac3ef3f73a79896160fee7e92d4bc6a18c71d)

Toolbar: move and rename filter to edit menu above Search

This is to expose more filter.
Also rename "Show Filter Bar" to simply "Filter", keep "Show Filter Bar" as tooltip

Adds a toggle filter bar action.

Merge branch 'release/20.12'

Re-allow to paste files via context menu

BUG: 429762

Utilize KFileItemActions::addActionsTo method from KIO

Remove unused includes

[kitemlistview]: Animate rubberband fading out

Add placeholder text for empty views

Throughout our QML-based software, we now display placeholder text for
empty views so that the user knows that the view is actually empty, and
it's not that the software is just broken.

This commit introduces the same thing for Dolphin's main view.

BUG: 429248
BUG: 426354
FIXED-IN: 21.04

ui.rc files: consistenly use <gui> instead of deprecated <kpartgui>

GIT_SILENT Upgrade release service version to 20.11.90.

Merge branch 'release/20.12'

Allow the openURLJob to run executables

Otherwise, excecutable files like AppImages won't be allowed to launch
even if they are correctly marked as executable.

BUG: 429603
FIXED-IN: 20.12

Fix typo in gitlab-ci filename

Add .gitlab-ci.yml

This should enable basic CI support for dolphin MRs.

Fix delete action switching via 'Shift' when sub-context-menu is open

Switching the 'Move to Trash' action to 'Delete' using the shift key
does not work when the mouse hovers a submenu. This fix resolves the
issue by using an event filter instead of the key event functions.

BUG: 425997

Move repetitive contextmenu related logic in one place

This makes a bit easier to handle the stuff.
Also we don't need the `addVersionControlPluginActions` utility method
and we can put it in the new utility method instead.

Merge branch 'release/20.12'

Places panel: highlight place only when it is displayed

BUG: 156678
FIXED-IN: 20.12

PlacesItemModelTest: consider also Pictures/Music/Videos folders

See a371e23b2b667bc85c311d04549f3d7348d00af5 in kio.

PlacesItemModelTest: use helper method to increase indexes

Instead of duplicating the logic all over the place.

Rename 'Url Navigator' to 'Location Bar'

Remove unneeded version check

Add 'Open Terminal Here' action from dolphin to menu

Select last visited folder when going up/back

This commit fixes a regression introduced by
ad5d3367c7fef4c3c11188a768f21a2ee2b3c025. A lot of connections were
changed there and at least one of them was wrong. The connections can
be separated into GUI connections and internal ones. This commit makes
it so the GUI stuff is connected to m_urlNavigatorConnected and
everything else to m_urlNavigator.

BUG: 429097

GIT_SILENT made messages (after extraction)

Merge branch 'release/20.12'

Re-allow icons that are not theme icon

Particularly concerned are .desktop files.

See also:
15baa93640bbb49162f26d439e006729ee9c3441

BUG: 429113

Allow having the UrlNavigators below the tab bar

This commit restores the possibility to have the UrlNavigators below
the tab bar. This will happen automatically whenever the UrlNavigator
is removed from the toolbar.

It is also now again possible to have the toolbar on the side. This
option is disabled while the toolbar contains the UrlNavigators.

This commit makes no changes to the new default which is having the
UrlNavigators in the toolbar but makes sure that upgrading users won't
be affected.

D-Bus activation systemd service

Correctly scopes dolphin in systemd service `app-org.kde.dolphin.service` when dbus-activated (while still working if systemd is not present), as per https://systemd.io/DESKTOP_ENVIRONMENTS/

uses new ECM modules:
- https://invent.kde.org/frameworks/extra-cmake-modules/-/merge_requests/5 to simplify the systemd service templating + install
- https://invent.kde.org/frameworks/extra-cmake-modules/-/merge_requests/17 to generate the dbus service file automatically

needs https://invent.kde.org/frameworks/extra-cmake-modules/-/merge_requests/25

Different icon to launch Konsole terminal

Separate Delete/Move To Trash menu items in context menu

Right now these actions are in a visual group with "Rename" and "Add to
Places" which is silly as those actions have no logical connection
whatsoever to deletion.

This commits moves those items into the section above, putting them in a
more logical place and leaving the Delete/Move To Trash item in its own
section.

BUG: 429256
FIXED-IN: 21.04

Add missing kconfig_version.h include

This should have been part of edced8460b.

Correct indentation and whitespace in prior commit

Allow folder creation on desktop via F10 shortcut

Merge branch 'release/20.12'

PlacesPanel: tooltip, use toLocalFile instead of path

To avoid Windows compatiblity issues.

Merge branch 'release/20.12'

Increment kpartgui version

The defaults changed in 37327c9b0aae112c5890703cba1f0157043007e0 but
I forgot to also bump the kpartgui version so this commit rectifies
that.

Context Menu: Only add paste action to folders

Either the viewport, or the selected folder is used as destination.
The paste action is only added when enabled.

GIT_SILENT made messages (after extraction)

Merge branch 'release/20.12'

Fix navigator alignment for right-to-left localizations

Merge branch 'release/20.12'

Fix qrc icon

Apparently the icon was not null, because the mimetype was known.
But there was no icon associated with it and we got an
icon which is not null, but has a null pixmap.

Set empty value for packagekit transaction flags

Otherwise the value would be Trusted Only, which will
forbid installing local packages

Set empty value for packagekit transaction flags

Otherwise the value would be Trusted Only, which will
forbid installing local packages

Merge branch 'release/20.12'

Places panel: show a tooltip after 500 ms

BUG: 426455
FIXED-IN: 20.12

Merge branch 'release/20.12'

Adress the third round of Angelaccio's review comments

Additionally remove some redundant code concerning UrlNavigator visuals.

Adress most of the second round of Angelaccio's review comments

This commit applies most suggestions which were made on the MR.
Most notably the DolphinUrlNavigator class is split up which leads to
the creation of a DolphinUrlNavigatorsController class.

Additionally some minor coding style and const correctness changes are
included.

The error value of cached integers is changed from -1 to INT_MIN
because situations could come up in which -1 would be a valid value.

Apply Méven Car's suggestions

Fix a crash and extract unrelated changes

The secondary UrlNavigator is now created when and only when:
- split view mode is activated for the active tab
OR
- switching to a tab that has split view already enabled.
This fixes a crash that occurs when the setting to always start in
split view mode is enabled.

An animation for activating split view is also removed from this and
moved into a separate MR. Another unrelated name change left over from
a previous commit (viewContainers() -> activeViewContainers()) is
dropped.

Make UrlNavigators in the toolbar the only option

The UrlNavigators will be automatically added to the toolbar. The Sort By
action is removed from the default toolbar to make space.

Remove all options to have UrlNavigators outside the toolbar and remove
those code paths.

Make it so the new NavigatorsWidgetAction contains two UrlNavigators when
in split view mode. Spacing was also added to align these UrlNavigators
with the ViewContainers when enough space is available.

Force the toolbar to be either at the top or bottom of the window.

Set a sane sizeHint for DolphinUrlNavigator. It would be better to do this
in KUrlNavigator in the future.

This commit also contains a changes which should be moved to a separate
merge requests before this gets merged:
- Add an expansion animation when split view is enabled by the user

Add the UrlNavigator to the toolbar automatically if needed

This commit adds the DolphinUrlNavigatorWidgetAction::addToToolbarAndSave()
method which changes the users toolbar configuration to contain an
Url Navigator. This way the user doesn't need to do anything manually.

Aside from that a bunch of minor fixes like renaming and reordering

Adress the first round of Angelaccio's review comments

- Split the viewContainers(bool includeInActive) into two methods
    without parameters
- Prevent users from accidently hiding all Url Navigators by
    preventing the dangerous action and then displaying a helpful
    message instead
Unrelated to review comments: Remove a useless line of code

trailing whitespaces

Add an option to use an UrlNavigator in the toolbar instead

This commit adds a locationInToolbar KToggleAction to switch between
using a location bar to navigate or using a new custom QWidgetAction
in the toolbar.

A big portion of this MR is refactoring because until now the
UrlNavigator was tightly intertwined with the DolphinViewContainer.
With this MR an UrlNavigator for controlling a View can be freely
connected or disconnected with a single method call.

A DolphinUrlNavigator class is created in the process which contains all
Dolphin-specific UrlNavigator code which did previously reside in the
DolphinViewContainer class. Other application parts that belong to
UrlNavigator-management are also moved here.

PlacesItemListWidget: Use Solid to check if device is a network share

Merge branch 'release/20.12'

Add missing reference

Iterate over a const copy list of containers

This effectively reverts the mutable iterations approach on
2448f88c5f42d7a2040fcf3bcd3c5f2a2f62cd03, and fix crashes
and ghost items when using the filter bar

BUG: 428374

GIT_SILENT Upgrade release service version to 21.03.70.

GIT_SILENT Upgrade release service version to 20.11.80.

Do not expose KCMs in application launchers

We still need to build them, because they
are also loaded on konqueror.

KDirectoryContentsCounter: fix support of symlinks dir

BUG: 428712
FIXED-IN: 20.12

Add support for KUserFeedback

This commit introduces KUserFeedback in dolphin with some basic data
sources and with a settings page to configure the telemetry values.

There are also a couple custom data sources as proof of concept: a bunch
of settings and the count of available network shares as listed by Solid.

The settings page is shown only if the user feedback framework is
enabled, but currently in Plasma we don't have a global kill switch to
disable it.

At the moment we never show an encouragement message. We need to connect
to the `Provider::showEncouragementMessage()` signal, but first we
should agree to a common way to show a non-annoying message to the users.

Details view: display empty text instead of unknown for sizeless dirs

BUG: 420037
BUG: 420040
FIXED-IN: 20.12

Move m_textInfo to fix ctor warning, and document why it's protected.

Use QDeadlineTimer instead of QDateTime.
Use early returns instead of deeply nested if/else.

Cleanup imports

Remove wip debugging

Fix freeSpaceInfo cache not being reset.

Use KDE GitLab url instead of the GitHub mirror

Cleanup unused colors. Use const. Use KColorScheme danger.

The draw code is more like the last state of D21312 now.

Refactor capacity by to use async KIO FreeSpaceJob.

Based on this kio filepicker patch:
https://github.com/KDE/kio/commit/933887dc334f3498505af7a86d25db7faae91019

Only show capacity bar for devices (has a udi).

Only check if place url is mount point once during polish event.

The capacity bars will no longer update when capacity changes. It only
checks the size on init.

Updating during polish event is only a temporarily solution just to
draft code flow for seperating logic from painting. Eventually this
logic update should be in a worker thread detached from blocking the
rendering thread that blocks launching the dolphin window.

Draw 2 Rectangles below mount points to represent a disk capacity bar

Draws overtop text that's goes below the baseline like a 'j'. It's
fairly difficult to add extra space below the text as the places view
is a "details" view, so the text is vertically centered in the
widget's "row" as you'd desire it to in the main file view.

We can make the widget 8px taller, adding 4px above and below so
there's adequate room for the capacitybar, but that wastes 4px for
each mount point.

Ideally we'd use the "compact" view, which displays the text/size in a
column beside the icon. However the compact view was not designed to
take up the entire width of the viewport. It's also designed to
overflow with a horizontal scrollbar.

Improve padding in the Places View

Revert "D-Bus activation systemd service"

This reverts commit eb441b11885fc44dcba1bef693b95819657ed753.

We'll push it again after we branch 20.12. We don't want to push a too
recent version of KF5 on distros.

!27

Compute text width taking into account the zoom level

This makes the icon view less crowded.

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

FolderPanel: prevents scanning directory tree recursively

KFileItemListView contents are periodically scanned by KFileItemModelRolesUpdater.
It uses then KDirectoryContentsCounter to scan directories to determine their size possibly recursively.

Introduce a scanDirectories setting to disable directory scanning by KFileItemModelRolesUpdater.

BUG: 426617
FIXED-IN: 20.08.3

GIT_SILENT made messages (after extraction)

Fix PlacesItemModelTest

Since 3dfa0c1c2f87bf1e6bc9 in kio timeline:// is no longer in the places
model.

servicemenuinstaller: use Q_GLOBAL_STATIC

Global statics should be avoided when possible.
This fixes a clazy warning.

Port away from deprecated QWheelEvent::delta()

QWheelEvent::angleDelta() returns a QPoint instead of int, so we need to
use its y() value to get the number of steps.

Use QString::rightRef() as suggested by clazy

make sure we use valid icons

Fix warnings about scaling pixmaps

GIT_SILENT Update Appstream for new release

(cherry picked from commit 92d41a743399f6f38ff55b9cb59657f5c5a5a510)

Port from QStandardPaths::DataLocation to QStandardPaths::AppDataLocation

They are the same on Linux (but DataLocation is deprecated). On Windows this enables the use of the roaming path.

NO_CHANGELOG

Disable session management for daemon

We don't need to autostart the daemon if it was running at logout.

BUG: 417219
FIXED-IN: 20.12

Fix wrong parameter in function call

In KDirectoryContentsCounterWorker::CountResult, the first parameter in
the call of walkDir was wrong. It should be path without conversion,
because the conversion is done in the called function.

BUG: 428282

Use mutable iterators where required

Compile with QT_NO_KEYWORDS