Delete obsolete class DolphinSettings

Fix issues with applying wrong view-mode

- The view properties dialog mixed the details-view with
  the compact-view (#285848).

- The view properties have been reset when applying a
  setting from the "Dolphin Preferences" dialog. The rootcause
  was that the obsolete DolphinSettings class implemented
  the saving in a wrong way because of the recent refactoring
  (will remove this class completely now to prevent such
  issues in future).

BUG: 285848
FIXED-IN: 4.8.0

Add Frank Reininghaus and Sebastian Trüg to the About-dialog

Fix layout-issues when sorting by roles that are changed

When sorting is enabled for a role that gets changed e.g. because
a value like the MIME-type is determined asynchronously, then the
layout might get messed up.

slotItemsMoved() has been adjusted to invalidate the sizehint-cache
and to update the group-headers.

BUG: 285542
FIXED-IN: 4.8.0

Don't use mixed units in size-column of details-view

This makes it tricky to compare the filesizes without adjusting
the sort-order, so now all sizes in the size-column are shown
in KiB or KB (dependent on the KLocale setting).

BUG: 219932
FIXED-IN: 4.8.0

Related fixes:
- Stay consistent with the rounding when using the KiB/KB unit
  in the statusbar.
- Fix sorting-by-size issue for folders
- Show "Unknown" in the size-column when the number of items
  cannot be determined.

Internal KFileItemModel optimizations and cleanups

- Use merge-sort instead of quick-sort. This assures a sane
  worst-case scenario where quick-sort has a runtime complexity of
  O(n*n) (e.g. when changing the sort-order from ascending to
  descending).

- lessThan()-improvements: Change internal data-structures to
  allow a comparison of any role, not only roles available
  in KFileItem

- Don't synchronously move an item if the value has been changed
  of a role defined as sort-role: This is too expensive in case if
  e.g. the sorting is done by "type" and the type is determined
  step by step.

Fix sorting-issues when value of a sort-role has been changed

If the value of a sort-role has been changed, emitting the signal
itemsChanged() is not sufficient as from the KItemModelBase point
of view an item has been moved or deleted/reinserted.

Corresponding to the unit-test KFileItemModel::setData() respects
this case correctly now, however there are some minor visual
animation issues left that (hopefully) should not be tricky to
solve.

Fix layout issue for groups

When enabling a group where the sort-role is retrieved
asynchronously (e.g. group by "type") the group-headers have
stayed invisible.

Fix grouping-issue with not visible sorting roles

It must be assured that the data for a sorting role always is
determined even it is not shown as "additional info" in the view.

Implement grouping for all roles

Fix minor visual issues in the view-engine

- Increasing the window-size should not result in increasing the
  size from previously invisible items
- Rename group 'Numerics' to '0 - 9'
- Fix "damaged" icons when changing the view-sizes or changing the
  view-mode

Group header fixes

- Simplify header for the horizontal scroll direction
- Fix issue that orientation change was not forwarded to the header

Fix visibility issues of group-headers

When expanding/collapsing folders in the tree-view the visibility
of group-headers got unpredictable.

Fix minor graphical issue in group header and items

Make group-headers less ugly

Improve group-header layout

- Apply a dynamic width
- Use a height that depends on the style
- Cleanup some interfaces and replace xxxBoundingRect() by
  xxxRect()

Mousewheel-support: Use smaller scroll-steps

Use only 1/4 of the scrollbar-page-size when the mousewheel
is used.

BUG: 284646
FIXED-IN: 4.8.0

Fix crash when grouping items

Because of a wrong usage of parent() vs. parentItem() a crash
could occur because the parent might be assigned to an already
deleted instance.

Implement grouping for names

- Use a custom header for KFileItems
- Cache the groups
- Allow enabling/disabling grouping dynamically

Currently there is a random crash in combination with the
groupheader-recycler, this will be fixed during the next
days.

Implement group-header layouting

The most tricky part for groups - the layouting - basically works
now for all views (grouping was available only in the icons views
for Dolphin 1.x) and is nearly equally fast as without groups.

Still open:
- Group headers are ugly screen rectangles
- Return valid groups in KFileItemModel instead of the currently
  hardcoded testing values
- Dynamically turning on/off groups does not work currently, the
  directory must be reentered

Sort the items in DolphinView::selectedItems(), part 2

I forgot this change in the last commit.
CCBUG:283409

Sort the items in DolphinView::selectedItems()

Before this commit, the order of the items in the list returned by
DolphinView::selectedItems() was random. Now the items are sorted like
they are sorted in the view.
BUG: 283409

Interface cleanups to prepare the return of "grouped sorting"

- Rename setCategorizedSorting() to setGroupedSorting()
- Change the model interface to allow enabling/disabling grouping
  without the need to declare a role (the sort role will be taken).
- Add dummy group role implementation in KFileItemModel

The grouping code itself requires some cleanups and might crash
at the moment or lead to weird layouts.

Fix regression concerning Control+mouse wheel zooming

Commit 9311f4bc97ec7000be46b8eec242a8 accidentally removed
DolphinView::wheelEvent(), which is responsible for the handling of
Control+mouse wheel events. These should change the zoom level.

Activate the DolphinView if the KItemListContainer gets focus

This is needed in the case that the view is split to make the
DolphinMainWindow aware of the view which is currently active.

Remove workaround to draw the focus indicator manually

In Oxygen the focus-indicator has only been drawn for widget
instances derived from QAbstractItemView and Q3ListView, so it
was not possible in Dolphin to use the style for drawing.

This has been fixed now in Oxygen, so using the style works now.

The custom code for drawing the text background will be removed
and replaced by a style-code later.

CCMAIL: hugo@oxygen-icons.org

Allow changing shortcuts for the Panel actions

Until now the shortcuts of the Panel actions could not be changed
as they did not show up in the "Configure shortcuts..." dialog.

The patch is based on the work of "SiegeLord" and Fabian Henze
attached to bug 273050.

BUG: 273050
FIXED-IN: 4.8.0

avoid two possible null pointer dereferences

move the setToolTip lines outside the if(action) inside it.
do not crash if there is no action.

Don't let KItemListView decide itself whether to show the header

The header should be turned on or off by the user of the
KItemListView classes.

Details-view: Fix column-width issue

The column-width must respect the minimum width of the
header-roles.

Remember the changed header-roles as part of the directory

Context menu cleanups

- Open the context menu on the mouse-press event, not on the
  mouse-release event.
- Provide an explicit position-information and don't use
  QCursor::pos(). This fixes the issue that opening a context-menu
  by the keyboard opens below the cursor.
- Provide different signals in the KItemListController for
  the different context-menu types (item vs. view vs. header).
- Implement turning on/off roles by the header-context-menu.

Assure that the rubberband is drawn above the items and not below

Provide alternating background colors for the details-view

Use QFontMetrics::height() instead of averageCharWidth()

QFontMetrics::averageCharWidth() is quite slow and does not work
well on fonts with broken characters. In the
usecases required by Dolphin QFontMetrics::height() is a better
and faster alternative to get a font-dependent minimum width.

Thanks go to Christoph Feck for this hint!

Don't apply an empty item-size to the layout

This fixes strange layout-issues when turning on/off previews in the
details-view.

CCBUG: 278899

Add function KItemListSelectionManager::isSelected(int index)

This function is used in KItemListView::updateWidgetProperties()
to find out if an item is selected in a more efficient way.
The new function is tested in KItemListSelectionManagerTest.
I've factored out some code from
KItemListSelectionManagerTest::testChangeSelection() to a new
member to simplify the test.

Remove TODO-comment which isn't needed anymore

Keep current item and selection when resorting, part 2

This commit adds the missing pieces to KItemListView and makes
KFileItemModel::resortAllItems() only emit the itemsMoved() signal.

Elide the texts if the user shrinks the column-widths

Fix layout when the header has been adjusted by the user

Fix itemsMoved() signal/slot signature: KItemRangeList -> KItemRange

Implement smooth-scrolling for horizontal and vertical scrollbars

Currently only a smoothscrolling was provided into the
scroll-direction, but not in case of the details-view where a
horizontal scrollbar might be shown too.

Some minor adjustments of the private KItemListSmoothScroller interface
will be done later...

Keep current item and selection when resorting, part 1

KFileItemModel now emits the itemsMoved signal when the model
is resorted, and KItemListSelectionManager has a new function
itemsMoved() which will be called indirectly when this signal
is emitted. Unit tests for the new functionality are included.
The following things are still needed to make the feature
work:
1. KFileItemMdel::resortAllItems() should not emit
   itemsAdded/itemsRemoved any more.
2. KItemListView::itemsMoved() must update the view according
   to the changes in the model, and it must call
   KItemListSelectionManager::itemsMoved().

Details view: Improve performance when expanding items

Prevent unnecessary calls to visibleRolesSizes() when
expanding items. Also the performance has been improved when
resizing the window.

Details view: Improve performance

When inserting items or when updating the item-roles there is no
need to recalculate the column-widths for all items to get an
optimized column-width.

SVN_SILENT made messages (.desktop file)

Provide scrollbar for large items

If an item does not fit into the available width/height a scrollbar
should be provided (e.g. typically this represents the horizontal
scrollbar in the details-view where the width can be larger than
the than the visible width).

Currently the interaction with the scrollbar is not implemented but
this will be a quite minor task in comparison to this patch.

Fix a spelling mistake in Dolphin.

Fixed selection of directories with a trailing slash used with --select
parameter.

As QHash would not match a KUrl key with it's value if the key had a
trailing slash, so it would return -1, hence the file won't get
selected.
Changed the UpdateViewState slot to remove the trailing slash before
calling the index function for the selected URL. Also modified
DolphinView to use KUrl list instead of KFileItemList to maintain
simplicity.

Update the layout asynchronously when items are changed

This results in less overhead if item-changes are done rather fast
(e.g. when applying the number of sub directories for the "size"
role).

Fix size-hint calculation in KFileItemListView

The current approach works fine but should be replaced in the longterm
by a helper class shared by KFileItemListWidget and
KFileItemListView to share information that is required to
calculate the size hints in KFileItemListView and to represent
the actual data in KFileItemListWidget.

Increase the width of the first column automatically

Additionally it is assured that on role-changes the size-hints
get updated if necessary.

Allow resizing of columns by the user

Still open:
- Column content is not clipped correctly
- First column is not automatically increased to the available
  width like in Dolphin 1.7

Update the layout periodically when resizing the window

Up to now the layout has only been updated when the geometry has
not been changed for at least 300 ms. This has been changed so
that at least each 300 ms the layout is updated to give a better
visual feedback when resizing the window.

KItemListHeader: Allow resizing and change of the sorting

KItemListView does not react on the resizing yet. Toggling the
sort order and changing the sort role is already possible.

Remove debugging output

Fixed bug for selecting files with --select parameter.

Files selected via --select parameter did not get current item focus.
And if files were deleted while being current item, updating the view
would select the next item instead of the first item in the list.

BUG: 257805
CCMAIL: peter.penz19@gmail.com
CCMAIL: frank78ac@googlemail.com

Improve drawing of list-header

Show the role-description in the header of the details view

Change signature of setVisibleRoles()

Simply use a QList<QByteArray> instead of QHash<QByteArray, int>.

Fixed documentation of KItemModelBase::itemsMoved()

Draw styled header for the details-view

Rough draft for getting back the header for the details-view

Implement 'Sort By Size'

It only works for files so far. The sorting of folders
by the number of items is more tricky to get right because
this number is retrieved asynchronously by
KFileItemModelRolesUpdater.

Fix crash in version-control-observer

The returned file-item of the root might be null.

BUG: 282108

Implement some missing sorting options

Changing the sort order and enabling/disabling the
"Sort Folders First" option works now.

Implement DolphinView::sorting()

This fixes the problem that changing the sorting criterion
from 'Name' to 'Date' and then back to 'Name' doesn't work.

Implement restoring expanded folders in Details View

Fix crash when expanding/closing a sub-tree

m_pendingItems and m_pendingInvisibleItems might contain already
removed items. Take care to delete them before starting to resolve
the roles.

Thanks to Frank Reininghaus for finding out the root-cause.

SVN_SILENT made messages (.desktop file)

Fix possible crash in KFileItemModel::slotRefreshItems()

If the changed items have been created recently, they might not be in
m_items yet.  In that case, the list 'indexes' might be empty, which
leads to a crash later on (I saw the crash in KFileItemModel's unit
test).

Provide KVersionControlPlugin version 2

Based on the work of Vishesh Yadav some extensions where required
for the KVersionControlPlugin interface that have found there way
now into KVersionControlPlugin2.

Beside some interface cleanups it is now possible that a version
control plugin may also provide context actions for directories or
files that are not versioned yet.

REVIEW: 102541

Improve cache handling in KItemListWidget

Provide a hook for derived KItemListWidget where the cache
can be updated.

Version control: Show context menu entries of plugins

Version control: Get rid of PendingThreadsMaintainer

SVN_SILENT made messages (.desktop file)

KFileItemModel: emit itemsChanged() signal

Version control: show pixmap overlays for the version state

Version control: Apply text-color if an item is versioned

SVN_SILENT made messages (.desktop file)

First step to reactivate version control plugin functionality

- Add a DolphinFileItemListWidget that provides icon-overlays
  and colored text for the version state (implementation is
  missing yet)
- Allow KFileItemListWidget to have custom text colors
- Update interface of VersionControlObserver to work with
  KFileItemModel instead of the old model-interface.

Improved drag and drop support

Simplified code for the viewmode settings

Improve autoscrolling

- Start autoscrolling when doing a dragging
- Start autoscrolling with a small delay

Preserve extensions when renaming multiple files

Thanks to Chiraq Anand for the patch! The commit is based on the
patch at https://git.reviewboard.kde.org/r/102495/ with some
minor modifications.

REVIEW: 102495

Removed warning, minor coding style update

Fix issue when moving items to trash

KDirLister does not emit a completed() signal when deleting files,
only when files are listed.

Items are expandable in the Details View.

Implement DolphinView::simplifiedSelectedUrls()

Makes it possible to trash and delete items.

KItemListKeyboardSearchManager improvements and unit tests

This commit implements a 'repeated key search' feature, similar
to QAbstractItemView, and adds unit tests for keyboard searching.

Made items a const.

DolphinView zoom with CTRL+MouseWheel REVIEW: 102490

Using selectedItems() method already present in DolphinView for slotItemActivated()

Fix context-menu selection issue

When requesting a context menu above a non-selected item, the
existing selection must be cleared.

Renamed some signals for consistency with KItemViews classes

Fixed a trivial bug that was breaking build.

Added the functionality to activate items by pressing 'enter' or 'return' key on them, equivalent to a mouse click.

REVIEW: 102450

Removed a warning of unused variable

Added support for highlighting items by typing their name on the keyboard.