| Age | Commit message (Collapse) | Author |
|
Constructing a KColorScheme object is very expensive because of a number
of tint computations. When scrolling a big list more than 30 % of the
time was spent here. Instead, we can precompute and store the inactive
text color. (see https://git.reviewboard.kde.org/r/100826/)
Thanks to Samuel Rødal for the patch!
|
|
|
|
When a searching has been triggered externally (e.g. by clicking on a tag) the search-mode settings like "From here"... are ignored and should not be shown at all. The current patch is only a quick workaround until a clean layout will be provided for this case.
|
|
The search panel must get a hint whether clicking on the facets should result in searching everywhere or from the current folder. It is not sufficient to check the search-settings of the "Find:"-box, as when the "Find:"-box is invisible there is no hint for the user what kind of searching is done and the setting must be ignored.
|
|
- Always merge the query from the "Find:"-field with the current facet-query. This assures that the facet-query never gets overwritten if the user changes the "Find:"-field.
- Always trigger a searching if the text of the "Find:"-field has been cleared.
Still a lot of usability issues are left, but this is at least a first step...
|
|
|
|
Thanks to Sebastian Dörner for the notification.
|
|
Bug reports indicate that users are confused by the term "filter" for this.
|
|
When Dolphin is started with a directory as argument unnecessary reload operations of KDirListers are done. The patch improves this by just letting the DolphinView internally do a loading as soon as it gets visible (and without reloading). Also in case if only one directory is passed as argument the current tab gets reused instead of creating new tabs and removing the current tab.
Thanks a lot to David Faure for pointing out this unefficient handling.
|
|
|
|
The new emblem is green, not red, which should be reflected by the text
color. Now, all files that will be part of the next commit
(LocallyModifiedVersion and AddedVersion) are green and unstaged files
darkGreen.
|
|
|
|
Commit 680009b522b2c9eda03201ac02dc18994b5a8bd0 resulted in a regression when switching from a view in the column-mode to a view in the icon- or details-mode: The wrong content has been shown. This patch reverts commit 680009b522b2c9eda03201ac02dc18994b5a8bd0 and solves the performance-issue in combination with previews by triggering the preview-generation in a queued way.
|
|
|
|
Adjust the preview-settings to allow users to configure thumbnail-plugins. For consistency also the service-settings have been adjusted to use the ServiceModel and ServiceItemDelegate.
|
|
This makes it unnecessary to manually apply the plugins. See https://git.reviewboard.kde.org/r/100578/ for more details.
|
|
If a change from a directory with disabled previews is done to a directory with enabled previews, also previews are generated for the previous directory as the preview-generator still contains the not updated directory lister. Because of this it is important to apply the view-properties after the directory-lister has been updated.
|
|
CCBUG: 265859
|
|
|
|
|
|
|
|
|
|
If only one file is selected, pressing RETURN should behave similar like triggering the item with the mouse. For this the signal itemTriggered() must be emitted.
CCBUG: 250475
|
|
CCBUG: 265714
|
|
Most developers seem to prefer
if (ptr) ...
if (!ptr) ...
in comparison to
if (ptr != 0) ...
if (ptr == 0) ...
Adjusted the Dolphin-code to use the "most-prefered style" to make contributors happy.
|
|
|
|
|
|
(cherry picked from commit 4caa285c065ea362ae52d93a567680513d8beb2a)
|
|
|
|
In this case the "From Here"/"Everywhere" buttons and the "Filename"/"Content" buttons are useless. Currently they just get disabled but the plan is to provide a better visual indication of the current query and to remove them completely.
|
|
CCBUG: 265547
|
|
|
|
A minor API cleanup in DolphinSearchBox has been done related to the test.
|
|
The clearing of the search-text should only be done when opening the search-box.
|
|
|
|
Also some KDE-includes have been capitalized
|
|
|
|
This fixes the issue that a directory might be declared as indexed/non-indexed although it just starts with a similar name like another directory.
|
|
|
|
|
|
The filter-panel should be disabled if the current folder is not indexed at all. Also when triggering a "Find" the filter-panel should stay invisible per default when the current folder is not indexed.
CCBUG: 264969
|
|
This fixes the issue that the initial size of the Filter Panel is too small.
|
|
|
|
CCBUG: 264791
|
|
CCBUG: 264796
|
|
|
|
svn path=/trunk/KDE/kdebase/apps/; revision=1217560
|
|
In particular, tests for changing the zoom level and for saving and
restoring the view state.
svn path=/trunk/KDE/kdebase/apps/; revision=1217270
|
|
Up to now, some Columns View tests that need a folder to be displayed
in the view only passed if they were run first: The first test always
passed, and if a test name was passed on the command line, such that
only this test would be run, this test would work as well. In other
cases, the view was empty, not showing the items in the folder.
The problem is worked around by creating a new DolphinView before each
test in the case of the Columns View. Added a TODO comment to make me
remember to look into this again at some point.
svn path=/trunk/KDE/kdebase/apps/; revision=1217260
|
|
The test assumes that the view does not have the keyboard focus
initially. However, this may not be the case if this is the only test
to be executed (i.e., if testKeyboardFocus is passed to the test
executable on the command line). This commit transfers the keyboard
focus explicitly to another widget.
svn path=/trunk/KDE/kdebase/apps/; revision=1217251
|