| Summary: | After deletion of selected files current item moves one down | ||
|---|---|---|---|
| Product: | [Applications] krusader | Reporter: | Peter Mühlenpfordt <devel> |
| Component: | general | Assignee: | Krusader Bugs Distribution List <krusader-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | krusader-bugs-null, toni.asensi |
| Priority: | NOR | ||
| Version First Reported In: | 2.6.0 | ||
| Target Milestone: | --- | ||
| Platform: | Ubuntu | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/krusader/dc38f3cfc6fd4477231967c03ea7fe76581d4e4c | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
| Attachments: | Panel before/after deletion of selected files | ||
Thanks, Peter. This didn't happen using Krusader 2.4.0-beta3. Note: This is related to
https://phabricator.kde.org/D10385
Git commit dc38f3cfc6fd4477231967c03ea7fe76581d4e4c by Peter Mühlenpfordt.
Committed on 14/02/2018 at 06:48.
Pushed by muhlenpfordt into branch 'master'.
Fix setting new current item after file deletion
Summary:
After deleting one or more selected files and current item is not one
of these, the current item is moved one down.
This patch modifies `KrView::firstUnmarkedBelowCurrent()` to not make
an additional step down in finding the next unselected item.
Test Plan:
1. Create a folder with 5 files "file1".."file5"
2. Select "file1" and "file2"
3. Set current item to "file3"
4. Press {key Del} and delete selected files
-> The current item should stay on "file3"
Reviewers: #krusader, martinkostolny
Reviewed By: #krusader, martinkostolny
Subscribers: martinkostolny, asensi
Differential Revision: https://phabricator.kde.org/D10385
M +4 -2 krusader/Panel/PanelView/krview.cpp
M +1 -1 krusader/Panel/PanelView/krview.h
M +2 -1 krusader/Panel/listpanel.cpp
https://commits.kde.org/krusader/dc38f3cfc6fd4477231967c03ea7fe76581d4e4c
|
Created attachment 110331 [details] Panel before/after deletion of selected files After deleting one or more selected files and current item is not one of these, the current item is moved one down. Steps to reproduce: 1. Create a folder with 5 files "file1".."file5" 2. Select "file1" and "file2" 3. Set current item to "file3" 4. Press Del and delete selected files Expected: Current item should stay on "file3" What happens: Current item is moved down to "file4" Tested with Krusader v2.6.0 from Kubuntu 17.10 packages and current master from git.