GitLab

Méven Car authored Nov 15, 2020 and Elvis Angelaccio committed Nov 19, 2020

Particularly concerned are .desktop files.

See also:
15baa936

BUG: 429113

Felix Ernst authored Nov 19, 2020 and Elvis Angelaccio committed Nov 19, 2020

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.

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:
- frameworks/extra-cmake-modules!5 to simplify the systemd service templating + install
- frameworks/extra-cmake-modules!17 to generate the dbus service file automatically

needs frameworks/extra-cmake-modules!25

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

This should have been part of edced846.

To avoid Windows compatiblity issues.

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

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

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.

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

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

BUG: 426455
FIXED-IN: 20.12

Felix Ernst authored Nov 09, 2020 and Elvis Angelaccio committed Nov 09, 2020

Additionally remove some redundant code concerning UrlNavigator visuals.

Felix Ernst authored Nov 05, 2020 and Elvis Angelaccio committed Nov 09, 2020

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.

Felix Ernst authored Oct 28, 2020 and Elvis Angelaccio committed Nov 09, 2020

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.

Felix Ernst authored Sep 20, 2020 and Elvis Angelaccio committed Nov 09, 2020

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

Felix Ernst authored Aug 27, 2020 and Elvis Angelaccio committed Nov 09, 2020

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

Felix Ernst authored Aug 12, 2020 and Elvis Angelaccio committed Nov 09, 2020

- 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

Felix Ernst authored Jun 14, 2020 and Elvis Angelaccio committed Nov 09, 2020

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.

Ismael Asensio authored Nov 06, 2020 and Elvis Angelaccio committed Nov 08, 2020

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

BUG: 428374